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

Revert "Merge PR #1772 into master" #1773

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

batrick
Copy link
Member

@batrick batrick commented Jun 23, 2022

Turtles.

Merge when Octopus is EOL. For reals.

See #1772 and #1624 for history.

@batrick batrick marked this pull request as draft June 23, 2022 14:19
This reverts commit 082ae7e, reversing
changes made to 31a356e.

Signed-off-by: Patrick Donnelly <[email protected]>
@batrick batrick requested a review from zmc March 28, 2024 19:49
@batrick batrick marked this pull request as ready for review March 28, 2024 19:51
@batrick batrick requested a review from idryomov March 28, 2024 19:51
Copy link
Contributor

@idryomov idryomov left a comment

Choose a reason for hiding this comment

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

See #1772 and #1624 for history.

I clicked through the links in the description of the PR, but I think it would be nicer if the history was actually explained in the commit message because right now it's a revert of a revert and commit 31a356e mentioned there isn't relevant today at all.

@idryomov
Copy link
Contributor

Also, while octopus is EOL, we are still testing upgrades from it to reef. Can this somehow interfere there (might be a dumb question but this had to be reverted once already so asking just to be sure)?

@batrick
Copy link
Member Author

batrick commented Mar 28, 2024

Also, while octopus is EOL, we are still testing upgrades from it to reef.

You mean quincy I assume?

Can this somehow interfere there (might be a dumb question but this had to be reverted once already so asking just to be sure)?

Ya maybe, let's wait one more release ;)

@batrick batrick marked this pull request as draft March 28, 2024 20:51
@idryomov
Copy link
Contributor

You mean quincy I assume?

Yes, apologies for the braino.

@kshtsk
Copy link
Contributor

kshtsk commented Jul 2, 2024

@batrick ping, do we still need this PR?

@batrick
Copy link
Member Author

batrick commented Jul 2, 2024

@batrick ping, do we still need this PR?

It's dead code to remove but needs to stick around for a while until we stop testing upgrade paths from octopus.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants