Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove multiple single quotes from Velero backup.status.validationErrors field #8245

Merged
merged 1 commit into from
Oct 9, 2024

Conversation

shubham-pampattiwar
Copy link
Collaborator

Thank you for contributing to Velero!

Please add a summary of your change

Remove multiple single quotes from error message

Please indicate you've done the following:

@shubham-pampattiwar shubham-pampattiwar added the kind/changelog-not-required PR does not require a user changelog. Often for docs, website, or build changes label Sep 24, 2024
@reasonerjt
Copy link
Contributor

@shubham-pampattiwar
Thanks but could you clarify why we wanna remove the single quote?

@shubham-pampattiwar
Copy link
Collaborator Author

@reasonerjt Validation error has extra quotes around BSL name:

status:
  expiration: "2024-10-27T17:54:55Z"
  formatVersion: 1.1.0
  phase: FailedValidation
  validationErrors:
  - 'an existing backup storage location wasn''t specified at backup creation time
    and the default ''sample-dpa-11'' wasn''t found. Please address this issue (see
    `velero backup-location -h` for options) and create a new backup. Error: BackupStorageLocation.velero.io
    "sample-dpa-11" not found'
  version: 1

With this change the extra quotes are removed:

status:
  expiration: "2024-10-27T18:20:38Z"
  formatVersion: 1.1.0
  phase: FailedValidation
  validationErrors:
  - 'an existing backup storage location wasn''t specified at backup creation time
    and the default sample-dpa-11 wasn''t found. Please address this issue (see `velero
    backup-location -h` for options) and create a new backup. Error: BackupStorageLocation.velero.io
    "sample-dpa-11" not found'

@kaovilai
Copy link
Contributor

wasn''t still looks bad

location wasn''t specified

…rs field

Signed-off-by: Shubham Pampattiwar <[email protected]>

update error message

Signed-off-by: Shubham Pampattiwar <[email protected]>
@shubham-pampattiwar
Copy link
Collaborator Author

wasn''t still looks bad

location wasn''t specified

Updated the error message:

validationErrors:
  - 'an existing backup storage location was not specified at backup creation time
    and the default sample-dpa-11 was not found. Please address this issue (see `velero
    backup-location -h` for options) and create a new backup. Error: BackupStorageLocation.velero.io
    "sample-dpa-11" not found'

Copy link

codecov bot commented Sep 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (main@11f771f). Learn more about missing BASE report.
Report is 8 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #8245   +/-   ##
=======================================
  Coverage        ?   59.21%           
=======================================
  Files           ?      367           
  Lines           ?    30841           
  Branches        ?        0           
=======================================
  Hits            ?    18262           
  Misses          ?    11119           
  Partials        ?     1460           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@reasonerjt reasonerjt merged commit db2eb89 into vmware-tanzu:main Oct 9, 2024
45 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
has-unit-tests kind/changelog-not-required PR does not require a user changelog. Often for docs, website, or build changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants