-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
e2e: add multiversion flag to generator #9829
Conversation
Do you mean this should be merged after all the other PRs you have open that're related to the E2E code? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
I mean that in terms of the multi-version feature, I don't foresee needing to make other changes to the Go code. The other e2e changes in my head are separate from this one and may require more changes to the Go. |
* add multiversion flag to generator * clarify flag comment (cherry picked from commit ac48630)
* add multiversion flag to generator * clarify flag comment (cherry picked from commit ac48630) # Conflicts: # test/e2e/generator/main.go
* add multiversion flag to generator * clarify flag comment (cherry picked from commit ac48630)
This pull request should be the last changes made directly to the Go e2e code. This adds a
multi-version
flag to the e2e test generator. When set with--multi-version <version-number>
the e2e generator will create manifests with with<version-number>
specified as the node version.The final piece of this group of changes will be setting this flag in the
v0.34.x
branch's invocation of thegenerator
tool. The flag will be set dynamically by querying git for the latestv0.34.*
tag.PR checklist
CHANGELOG_PENDING.md
updated, or no changelog entry neededdocs/
) and code comments, or nodocumentation updates needed