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.

Sign up for GitHub

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

Jump to bottom

bump java sdk and k8s versions #6044

Merged
merged 3 commits into from Oct 17, 2022
Merged

bump java sdk and k8s versions #6044

merged 3 commits into from Oct 17, 2022

Conversation

laxmikantbpandhare
Copy link
Member

@laxmikantbpandhare laxmikantbpandhare commented Sep 27, 2022 •
edited

Description of the change:
Update java-operator-plugins with new changes.

Motivation for the change:
fix issue #6031

Checklist

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

@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy September 27, 2022 17:53 Inactive
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy September 27, 2022 17:53 Inactive
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy September 27, 2022 17:53 Inactive
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy September 27, 2022 17:53 Inactive
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy September 27, 2022 17:53 Inactive
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy September 27, 2022 17:53 Inactive
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy September 27, 2022 17:53 Inactive
@laxmikantbpandhare laxmikantbpandhare changed the title udpated java version to get new changest udpated java version to get new changes Sep 27, 2022
@laxmikantbpandhare laxmikantbpandhare self-assigned this Sep 27, 2022
@laxmikantbpandhare laxmikantbpandhare added this to the v1.24.0 milestone Sep 27, 2022
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy September 27, 2022 18:00 Inactive
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy September 27, 2022 18:00 Inactive
@laxmikantbpandhare
Copy link
Member Author

/hold

@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 Sep 27, 2022
@laxmikantbpandhare
Copy link
Member Author

laxmikantbpandhare commented Sep 27, 2022 •
edited

Other k8s depedencies also got bumped which is not expected as part of this PR.

@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy September 27, 2022 18:09 Inactive
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy September 27, 2022 18:09 Inactive
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy September 27, 2022 18:10 Inactive
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy September 27, 2022 18:11 Inactive
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy September 27, 2022 18:11 Inactive
everettraven
everettraven approved these changes Sep 27, 2022
View reviewed changes
Copy link
Contributor

@everettraven everettraven left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Sep 27, 2022
everettraven
everettraven reviewed Sep 28, 2022
View reviewed changes
go.mod Outdated Show resolved Hide resolved
@jmrodri
Copy link
Member

jmrodri commented Oct 7, 2022

/hold

@jberkhahn jberkhahn modified the milestones: v1.24.0, v1.25.0 Oct 7, 2022
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy October 17, 2022 14:34 Inactive
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy October 17, 2022 14:34 Inactive
@laxmikantbpandhare
rebase
8f2325b 
Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com>

updated changelog

Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com>

missong go.sum entry for controller runtime

updated sdk bumps

Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com>
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy October 17, 2022 14:40 Inactive
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy October 17, 2022 14:40 Inactive
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy October 17, 2022 14:40 Inactive
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy October 17, 2022 14:41 Inactive
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy October 17, 2022 14:44 Inactive
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy October 17, 2022 14:44 Inactive
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy October 17, 2022 14:45 Inactive
@laxmikantbpandhare
added changelog
6be9f90 
Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com>
@laxmikantbpandhare
added changelog
f610a5b 
Signed-off-by: laxmikantbpandhare <laxmikantpandhare@gmail.com>
@laxmikantbpandhare
Copy link
Member Author

/hold cancel

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 17, 2022
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy October 17, 2022 14:50 Inactive
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy October 17, 2022 14:50 Inactive
jmrodri
jmrodri approved these changes Oct 17, 2022
View reviewed changes
Copy link
Member

@jmrodri jmrodri left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Oct 17, 2022
everettraven
everettraven approved these changes Oct 17, 2022
View reviewed changes
Copy link
Contributor

@everettraven everettraven left a comment

Choose a reason for hiding this comment

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

/lgtm

@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy October 17, 2022 14:57 Inactive
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy October 17, 2022 14:57 Inactive
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy October 17, 2022 14:58 Inactive
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy October 17, 2022 14:59 Inactive
@laxmikantbpandhare laxmikantbpandhare temporarily deployed to deploy October 17, 2022 15:00 Inactive
@laxmikantbpandhare laxmikantbpandhare merged commit a5e0e28 into operator-framework:master Oct 17, 2022
@laxmikantbpandhare laxmikantbpandhare mentioned this pull request Oct 17, 2022
Closed
@laxmikantbpandhare laxmikantbpandhare linked an issue Oct 17, 2022 that may be closed by this pull request
Quarkus Operator SDK version is outdated #6031
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jmrodri jmrodri jmrodri approved these changes

@everettraven everettraven everettraven approved these changes

@theishshah theishshah Awaiting requested review from theishshah

Assignees

@jmrodri jmrodri

@everettraven everettraven

@laxmikantbpandhare laxmikantbpandhare

Labels
lgtm Indicates that a PR is ready to be merged. release-blocker This issue blocks the parent release milestone
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Quarkus Operator SDK version is outdated
5 participants
@laxmikantbpandhare @jmrodri @theishshah @everettraven @jberkhahn