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

rebase: update golang version to 1.17.5 #2703

Merged
merged 2 commits into from
Dec 10, 2021
Merged

Conversation

Madhu-1
Copy link
Collaborator

@Madhu-1 Madhu-1 commented Dec 10, 2021

This commit updates the golang to the latest available release i.e 1.17.5

Updating golang to 1.7.x will unblock #2699

Signed-off-by: Madhu Rajanna madhupr007@gmail.com

@mergify mergify bot added the rebase update the version of an external component label Dec 10, 2021
@Madhu-1 Madhu-1 requested review from nixpanic and a team December 10, 2021 05:08
@Madhu-1 Madhu-1 added Backport This PR represents it a backport from master to release branch backport-to-release-v3.4 and removed Backport This PR represents it a backport from master to release branch labels Dec 10, 2021
@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Dec 10, 2021

adding backport label as we need to support 3.4 for a while

@nixpanic
Copy link
Member

adding backport label as we need to support 3.4 for a while

Could you explain why this is needed for 3.4? We usually don't backport features, only bugfixes...

Copy link
Member

@nixpanic nixpanic left a comment

Choose a reason for hiding this comment

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

I think the golang version in go.mod needs to be adjusted too?

@nixpanic
Copy link
Member

I think the golang version in go.mod needs to be adjusted too?

If you want to keep that as part of #2699, that is fine too.

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Dec 10, 2021

I think the golang version in go.mod needs to be adjusted too?

not a blocker for now. will adjust when updating the go.mod even with 1.16 in go.mod everything works fine.

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Dec 10, 2021

I think the golang version in go.mod needs to be adjusted too?

If you want to keep that as part of #2699, that is fine too.

yes will adjust it as part of #2699

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Dec 10, 2021

adding backport label as we need to support 3.4 for a while

Could you explain why this is needed for 3.4? We usually don't backport features, only bugfixes...

yes, correct this is not needed. dropping it

@Madhu-1 Madhu-1 requested a review from a team December 10, 2021 07:36
@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Dec 10, 2021

@Mergifyio rebase

This commit updates the golang to the
latest available release i.e 1.17.5

Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
adding protobuf to go.mod to fix
`from implicitly required module
to add missing requirement` error

Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
@mergify
Copy link
Contributor

mergify bot commented Dec 10, 2021

rebase

✅ Branch has been successfully rebased

@Madhu-1 Madhu-1 added the ci/retry/e2e Label to retry e2e retesting on approved PR's label Dec 10, 2021
@mergify mergify bot merged commit 42403e2 into ceph:devel Dec 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/retry/e2e Label to retry e2e retesting on approved PR's rebase update the version of an external component
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants