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

Upgrade deps with go 1.17 #5505

Merged
merged 5 commits into from
Jan 19, 2022
Merged

Conversation

camilamacedo86
Copy link
Contributor

@camilamacedo86 camilamacedo86 commented Jan 18, 2022

Description of the change:
Upgrade deps with go 1.17:

  • k8s 1.23.1
  • controller-runtime 0.11,
  • Kubebuilder latest release v3.3.0
  • operator-lib to 0.10
  • operator-framework/api
  • controller-tools v0.8.0
  • operator-registry v1.19.5

Motivation for the change:

Closes: #5503

Checklist

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

@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 18, 2022
@openshift-ci openshift-ci bot requested review from jmrodri and theishshah January 18, 2022 14:08
@camilamacedo86 camilamacedo86 force-pushed the bump-sdk branch 4 times, most recently from bf4d979 to ff1567c Compare January 18, 2022 22:20
@camilamacedo86 camilamacedo86 changed the title upgrades Upgrade deps with go 1.17 Jan 18, 2022
go.mod Show resolved Hide resolved
)

exclude github.com/spf13/viper v1.3.2 // Required to fix CVE-2018-1098
Copy link
Contributor Author

Choose a reason for hiding this comment

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

it is no longer added

// Using containerd 1.4.0+ resolves an issue with invalid error logging
// from an init function in containerd. This replace can be removed when
// one of our direct dependencies begins using containerd v1.4.0+
github.com/containerd/containerd => github.com/containerd/containerd v1.4.3
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@camilamacedo86 camilamacedo86 marked this pull request as ready for review January 19, 2022 00:48
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 19, 2022
- fabianvf
- jberkhahn
Copy link
Member

Choose a reason for hiding this comment

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

Should this be done in a separate PR?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@@ -12,7 +12,6 @@ approvers:
reviewers:
- asmacdo
- camilamacedo86
- estroz
Copy link
Member

Choose a reason for hiding this comment

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

This too, in a separate PR maybe?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I am not sure how this file is working but it is only updated because was falling in the sanity. I updated with the sanity to make it pass.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It shows that needs to be updated because of : https://github.com/operator-framework/operator-sdk/pull/5483/files.

changelog/fragments/upgrades.yaml Outdated Show resolved Hide resolved
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 Jan 19, 2022
Copy link
Member

@varshaprasad96 varshaprasad96 left a comment

Choose a reason for hiding this comment

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

/lgtm

@asmacdo
Copy link
Member

asmacdo commented Jan 19, 2022

/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 Jan 19, 2022
Copy link
Member

@asmacdo asmacdo left a comment

Choose a reason for hiding this comment

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

/lgtm

@asmacdo asmacdo removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 19, 2022
@asmacdo asmacdo merged commit f950211 into operator-framework:master Jan 19, 2022
@camilamacedo86 camilamacedo86 deleted the bump-sdk branch January 20, 2022 14:04
@ryantking ryantking mentioned this pull request Mar 16, 2022
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants