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

Sync v2-5-stable with v2-5-test to release 2.5.1 #28997

Merged
merged 4 commits into from
Jan 18, 2023
Merged

Conversation

pierrejeambrun
Copy link
Member

@pierrejeambrun pierrejeambrun commented Jan 17, 2023

Add fix for #28988.

To prepare 2.5.1rc2

@boring-cyborg boring-cyborg bot added area:UI Related to UI/UX. For Frontend Developers. area:webserver Webserver related Issues labels Jan 17, 2023
@potiuk
Copy link
Member

potiuk commented Jan 17, 2023

I think you might also want to add #28962

…28962)

The dnspython has been released 2 hours ago and apparently eventlet
that cassandra-driver uses is not compatible with the new version.

Until the problem is fixed, we should limit dnspython to <2.3.0.

Related: eventlet/eventlet#781
(cherry picked from commit d169df7)
@pierrejeambrun pierrejeambrun added the type:misc/internal Changelog: Misc changes that should appear in change log label Jan 17, 2023
@pierrejeambrun
Copy link
Member Author

pierrejeambrun commented Jan 17, 2023

Right, I forgot about that, thank you. I just added it :).

@pierrejeambrun pierrejeambrun removed the type:misc/internal Changelog: Misc changes that should appear in change log label Jan 17, 2023
@potiuk
Copy link
Member

potiuk commented Jan 18, 2023

As discussed in #29004 - eventlet released a fix and we should cherry-pick (now merged) #29004 on top of the #28962 as this is a better long term fix.

The eventlet/eventlet#782 fixed error that
caused the need to limit dnspython and eventlet 0.33.3 has been
released. We can now remove the limit
for dnspython as long as we also limit eventlet to >=0.33.3.

(cherry picked from commit eaf5675)
@potiuk
Copy link
Member

potiuk commented Jan 18, 2023

The failing "check that image builds quickly" is fine and we can ignore it. The image will not build quickly now until we update constraints because the eventlet>=0.33.3 will conflict with the current constraints.

This should fix itself after constraints will get updated (and it should succeed when we re-run it after this build gets merged and constraints will be pushed).

@ephraimbuddy ephraimbuddy self-assigned this Jan 18, 2023
@potiuk
Copy link
Member

potiuk commented Jan 18, 2023

Ah I see that it even works now :)

@pierrejeambrun pierrejeambrun merged commit 49867b6 into v2-5-stable Jan 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:UI Related to UI/UX. For Frontend Developers. area:webserver Webserver related Issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants