-
Notifications
You must be signed in to change notification settings - Fork 44
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
Ansible 6.5.0: Dependencies, changelog and porting guide #169
Ansible 6.5.0: Dependencies, changelog and porting guide #169
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
444a40c
to
b55c4dd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks for working on this @rooftopcellist
@@ -0,0 +1,107 @@ | |||
_ansible_version: 6.5.0 | |||
_ansible_core_version: 2.13.5 | |||
amazon.aws: 3.5.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm missing the _python
entry that would be added by antsibull 0.51.0. I think it's ok to do the current release as-is, but please make sure to use the latest antsibull release for the next Ansible release :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@felixfontein I used the latest version originally, but CI complained about the python3.9 line, so I reverted back to 0.49.0, which was right before this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change I think: ansible-community/antsibull-build#448
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, that's helpful to know. There seems to be a bug in there, which for some reason CI didn't complain about so far. I'll try to look at that today or tomorrow. If anyone else is interested, the error is here: https://github.com/ansible-community/ansible-build-data/actions/runs/3236677877/jobs/5302816585#step:10:98
I'm a bit confused over the timing, though. Shouldn't this release have been made a week ago? (And 6.6.0 will then happen in two weeks?) |
You are right, this should have gone out a week ago as per the discussion on irc. This slipped, partially because of miscommunication and partially because of workload, which is unfortunate. I'll merge this for now, but let's chat more in IRC about the best way to move forward schedule-wise for 6.x releases and make sure we have the same plan in mind for the schedule for 7.x releases (as there has been some discussion in IRC around moving to a 4 week release cadence to be a little more aligned with the ansible-core release schedule. |
No description provided.