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

(actions): bump go version for go e2e tests #5841

Closed

Conversation

everettraven
Copy link
Contributor

Description of the change:
Make Go e2e tests run with Go version 1.18

Motivation for the change:
As part of the Kubebuilder bump to support k8s 1.24, Go 1.18 is now used when scaffolding a project with the go/v3 plugin. We need to adjust our e2e tests/actions to accommodate this change in the scaffolding.

Checklist

If the pull request includes user-facing changes, extra documentation is required:

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
@everettraven everettraven changed the title Actions/bump go e2e (actions): bump go version for e2e tests Jun 8, 2022
@everettraven everettraven changed the title (actions): bump go version for e2e tests (actions): bump go version for go e2e tests Jun 8, 2022
@everettraven
Copy link
Contributor Author

/hold

@everettraven everettraven added release-blocker This issue blocks the parent release milestone blocked labels Jun 8, 2022
@everettraven everettraven added this to the v1.22.0 milestone Jun 8, 2022
@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jun 8, 2022
@@ -0,0 +1,18 @@
# entries is a list of entries to include in
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do we need a changelog?
How do bump the ci tests affect who uses the SDK CLI tool?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point. I think I added it without thinking, will remove it. However this PR may get closed and this step be done as part of #5843

@everettraven
Copy link
Contributor Author

Closing as obsolete. Tests and testdata will be updated as a part of #5843

@everettraven everettraven removed release-blocker This issue blocks the parent release milestone blocked do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. labels Jun 9, 2022
@everettraven everettraven deleted the actions/bump-go-e2e branch June 9, 2022 13:49
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.

None yet

2 participants