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

Do not process gcloudignore files when uploading support-packages #1818

Merged
merged 1 commit into from
Sep 3, 2024

Conversation

Molter73
Copy link
Collaborator

@Molter73 Molter73 commented Aug 30, 2024

Description

This is a minor change, we are getting some error message on the upload support packages steps from the CPaaS workflows because the google-github-actions/upload-cloud-storage action expects a gcloudignore to be present by default. Since we don't expect to process this kind of file, we just set the option to false on all occurrences of the action.

Without this change the actions still worked correctly, this is merely to get rid of the annoying error messages in the CI output.

Examples of the errors can be found here: https://github.com/stackrox/collector/actions/runs/10609617180/job/29405734430

Checklist

  • Investigated and inspected CI test results
  • Updated documentation accordingly

Automated testing

  • Added unit tests
  • Added integration tests
  • Added regression tests

If any of these don't apply, please comment below.

Testing Performed

  • Run CPaaS steps and check the messages are gone.

This is a minor change, we are getting some error message on the upload
support packages steps from the CPaaS workflows because the
google-github-actions/upload-cloud-storage action expects a gcloudignore
to be present by default. Since we don't expect to process this kind of
file, we just set the option to false on all occurrences of the action.

Without this change the actions still worked correctly, this is merely
to get rid of the annoying error messages in the CI output.
@Molter73 Molter73 requested a review from a team as a code owner August 30, 2024 10:01
@Molter73 Molter73 added run-cpaas-steps Trigger CI steps related to cPaaS skip-integration-tests labels Aug 30, 2024
@Molter73 Molter73 merged commit f0acc8a into master Sep 3, 2024
73 of 103 checks passed
@Molter73 Molter73 deleted the mauro/do-not-process-gcloudignore branch September 3, 2024 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
run-cpaas-steps Trigger CI steps related to cPaaS skip-integration-tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants