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

Fix zipp version to fix ST2 CI issues. #6015

Merged
merged 2 commits into from Aug 19, 2023
Merged

Fix zipp version to fix ST2 CI issues. #6015

merged 2 commits into from Aug 19, 2023

Conversation

amanda11
Copy link
Contributor

@amanda11 amanda11 commented Aug 18, 2023

Current CI builds are failing when performing the st2client_install_check, see https://github.com/StackStorm/st2/actions/runs/5897223026/job/15996511713

Changes from when it first failed were zipp version got incremented, and that is where error is.

Fixed zipp version in requirements used by st2client and re-generated requirements.txt files and pants lock files.

Ran make .st2client_install_check manually to ensure that it now passes and pulls in a valid zipp version.

@pull-request-size pull-request-size bot added the size/XXL PR that changes 1000+ lines. You should absolutely split your PR into several. label Aug 18, 2023
@amanda11 amanda11 marked this pull request as draft August 18, 2023 13:15
@amanda11 amanda11 changed the title Fix zipp version to fix ST2 CI issues. [WIP] Fix zipp version to fix ST2 CI issues. Aug 18, 2023
@amanda11 amanda11 added this to the 3.9.0 milestone Aug 18, 2023
@amanda11 amanda11 changed the title [WIP] Fix zipp version to fix ST2 CI issues. Fix zipp version to fix ST2 CI issues. Aug 18, 2023
@amanda11 amanda11 marked this pull request as ready for review August 18, 2023 13:34
@amanda11 amanda11 requested a review from a team August 18, 2023 14:12
@cognifloyd cognifloyd merged commit ab2cc34 into master Aug 19, 2023
28 checks passed
@cognifloyd cognifloyd deleted the pin_zipp branch August 19, 2023 00:36
jk464 pushed a commit to jk464/st2 that referenced this pull request Aug 23, 2023
jk464 pushed a commit to jk464/st2 that referenced this pull request Aug 23, 2023
@arm4b arm4b mentioned this pull request Oct 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/XXL PR that changes 1000+ lines. You should absolutely split your PR into several.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants