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

Update from Mongo 4.4 to Mongo 7 in CI #6246

Merged
merged 6 commits into from
Sep 26, 2024
Merged

Update from Mongo 4.4 to Mongo 7 in CI #6246

merged 6 commits into from
Sep 26, 2024

Conversation

cognifloyd
Copy link
Member

@cognifloyd cognifloyd commented Sep 19, 2024

Switches from Mongo 4.4 to Mongo 7.0 using work by @guzzijones cherry-picked from #6236.

Also updates the redis container definition in the orquesta workflow to be consistent with ci workflow changes added in #6245.

@cognifloyd cognifloyd added this to the 3.9.0 milestone Sep 19, 2024
@cognifloyd cognifloyd self-assigned this Sep 19, 2024
@pull-request-size pull-request-size bot added size/M PR that changes 30-99 lines. Good size to review. size/XL PR that changes 500-999 lines. Consider splitting work into several ones that easier to review. and removed size/M PR that changes 30-99 lines. Good size to review. labels Sep 19, 2024
@cognifloyd cognifloyd changed the title CI: Switch from Mongo 4.4 to Mongo 7.0 Update to Mongo 7, mongoengine 0.29, and pymongo 4.6 Sep 20, 2024
@cognifloyd cognifloyd requested a review from a team September 20, 2024 04:19
@cognifloyd cognifloyd marked this pull request as ready for review September 20, 2024 04:20
@pull-request-size pull-request-size bot added size/XXL PR that changes 1000+ lines. You should absolutely split your PR into several. and removed size/XL PR that changes 500-999 lines. Consider splitting work into several ones that easier to review. labels Sep 20, 2024
@cognifloyd

This comment was marked as resolved.

conf/st2.conf.sample Outdated Show resolved Hide resolved
@guzzijones guzzijones self-requested a review September 22, 2024 14:55
@guzzijones
Copy link
Contributor

I would rather not split this out. we can do the pymongo update in a separate pr in the future to avoid the debug log flood. I am guessing we can filter those out with pytest anyway.

@cognifloyd cognifloyd marked this pull request as draft September 24, 2024 03:29
@cognifloyd

This comment was marked as outdated.

@cognifloyd

This comment was marked as outdated.

@pull-request-size pull-request-size bot added size/M PR that changes 30-99 lines. Good size to review. and removed size/XXL PR that changes 1000+ lines. You should absolutely split your PR into several. labels Sep 26, 2024
@cognifloyd cognifloyd marked this pull request as ready for review September 26, 2024 03:25
@cognifloyd
Copy link
Member Author

I just rebased this on master now that #6250 and #6252 have been merged. This PR is now focused on Makefile / CI updates to support MongoDB 7. So, this is ready for review.

@cognifloyd cognifloyd changed the title Update to Mongo 7, mongoengine 0.29, and pymongo 4.6 Update from Mongo 4.4 to Mongo 7 in CI Sep 26, 2024
@cognifloyd cognifloyd merged commit e02aec0 into master Sep 26, 2024
31 checks passed
@cognifloyd cognifloyd deleted the mongo7 branch September 26, 2024 05:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infrastructure: ci/cd mongodb size/M PR that changes 30-99 lines. Good size to review. tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants