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

ci: do not use --dirty when generating CI image version and add upgrade job as part of CI #136

Merged
merged 2 commits into from
Aug 6, 2021
Merged

Conversation

chewong
Copy link
Contributor

@chewong chewong commented Aug 6, 2021

Reason for Change:

kustomize edit set image causes the git branch to be in a dirty state, which causes the subsequent e2e test to use a non-existing webhook image.

Requirements

  • squashed commits
  • included documentation
  • added unit tests and e2e tests (if applicable).

Issue Fixed:

Fixes #135

Please answer the following questions with yes/no:

Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?

  • yes
  • no

Notes for Reviewers:

@chewong chewong changed the title ci: do not use --dirty when generating CI image version … ci: do not use --dirty when generating CI image version Aug 6, 2021
@chewong chewong marked this pull request as ready for review August 6, 2021 20:13
@chewong chewong requested a review from aramase August 6, 2021 20:13
@chewong chewong changed the title ci: do not use --dirty when generating CI image version ci: do not use --dirty when generating CI image version and add upgrade job as part of CI Aug 6, 2021
@chewong chewong enabled auto-merge (squash) August 6, 2021 20:58
@chewong chewong merged commit b7446c0 into Azure:main Aug 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

upgrade_* tests failing in the nightly run
2 participants