New collection: ravendb.ravendb #80
Replies: 15 comments 16 replies
-
@TheGoldenPlatypus hello, thanks for submitting the collection! Someone from @ansible-collections/steering-committee will take a look as soon as they have time. Please be patient, thanks |
Beta Was this translation helpful? Give feedback.
-
Ansible Collections Checklist (short version)For details about the following points, refer to the Collection Requirements. Every comment should say whether the reviewer expects it to be addressed, or whether it's optional. Note for reviewers: If you don't know how to check any of the points below, please ask maintainers of the collection you're reviewing or a Steering Committee member for clarifications in comments of corresponding inclusion discussion. Public availability and communication:
Standards and documentation:
Collection management:
Tests: Note for reviewers: If you don't know how to check the points below, please ask maintainers of the collection you're reviewing how you can do it.
|
Beta Was this translation helpful? Give feedback.
-
Ansible Collections Checklist (short version)For details about the following points, refer to the Collection Requirements. Every comment should say whether the reviewer expects it to be addressed, or whether it's optional. Note for reviewers: If you don't know how to check any of the points below, please ask maintainers of the collection you're reviewing or a Steering Committee member for clarifications in comments of corresponding inclusion discussion. Public availability and communication:
Standards and documentation:
Collection management:
Tests: Note for reviewers: If you don't know how to check the points below, please ask maintainers of the collection you're reviewing how you can do it.
|
Beta Was this translation helpful? Give feedback.
-
Ansible Collections Checklist (short version)For details about the following points, refer to the Collection Requirements. Every comment should say whether the reviewer expects it to be addressed, or whether it's optional. Note for reviewers: If you don't know how to check any of the points below, please ask maintainers of the collection you're reviewing or a Steering Committee member for clarifications in comments of corresponding inclusion discussion. Public availability and communication:
Standards and documentation:
Collection management:
Tests: Note for reviewers: If you don't know how to check the points below, please ask maintainers of the collection you're reviewing how you can do it.
|
Beta Was this translation helpful? Give feedback.
-
@ansible-collections/steering-committee : Second short review is needed here. i've conducted the first inclusion review and approved the collection.
@ansible-collections/steering-committee please conduct the review ^ and explicitly approve the collection. |
Beta Was this translation helpful? Give feedback.
-
@TheGoldenPlatypus hey, i got feedback from another SC member:
Could you please take a look? |
Beta Was this translation helpful? Give feedback.
-
@TheGoldenPlatypus i've created the inclusion topic https://forum.ansible.com/t/default-date-2025-08-04-collection-inclusion-include-ravendb-ravendb-in-the-ansible-community-package/44150, FYI If no objections, the collection will get included next week, i.e. will appear in the next minor Ansible package release. |
Beta Was this translation helpful? Give feedback.
-
@TheGoldenPlatypus FYI I don't have any objections to include the collection, only about the suggested date when to do it. See my comment on the forum. So it's possible that you have to wait until 12.1.0 (ETA 2025-08-19) to see it in the Ansible Community Package. |
Beta Was this translation helpful? Give feedback.
-
Hi everyone, I've already published the updated version on the Galaxy platform. Also, is there any documentation or guidance on what happens after inclusion? Thanks ! |
Beta Was this translation helpful? Give feedback.
-
When looking at the tests, I saw that you're running (Check out https://docs.ansible.com/ansible/devel/community/collection_contributors/collection_requirements.html#ci-testing.) |
Beta Was this translation helpful? Give feedback.
-
@TheGoldenPlatypus, looking at the CI configuration, I see that the integration tests and unit tests are only running with a single version of Ansible and Python, and notably, not running against ansible-core 2.19 which will be included in Ansible 12. It is a strict requirement that collections run CI tests against all versions of Ansible Core included in currently maintained versions of the ansible package (see the link @felixfontein included). Many collections use |
Beta Was this translation helpful? Give feedback.
-
@felixfontein @gotmax23, I’m going to be super verbose here since we’ve got a bunch of Python and Ansible versions in play, plus some best practice stuff. Easy to get lost in it all, so heads up :) I’ve updated our GH Actions CI to make Python / Ansible compatibility explicit, so we cover all supported combinations. 1. Sanity Testsbased on @felixfontein comment, I've changed the so for example - Running sanity test "compile" on Python 2.7
...
Running sanity test "compile" on Python 3.6
...
Running sanity test "compile" on Python 3.7
...
Running sanity test "compile" on Python 3.8
...
Running sanity test "compile" on Python 3.9
...
Running sanity test "compile" on Python 3.10
...
Running sanity test "compile" on Python 3.11
...
Running sanity test "compile" on Python 3.12 so according to our CI matrix
the following combinations will be tested:
2. Integration + Unit Testsfor Molecule scenarios and unit tests things work differently: molecule runs on the host Python, not in a
I’m fairly confident I’ve got full coverage here, but I wanted to lay out the details so we’re all on the same page. I’d love to hear your thoughts. I dropped Python 2.7 for stable-2.16 in the integration tests because it doesn’t play well with here’s the PR with the changes: link |
Beta Was this translation helpful? Give feedback.
-
@gotmax23 first, I had to deal with the "Molecule + Ansible 2.19/devel conditional issue" : ansible-community/molecule-plugins#311 second, I rewrote parts of the role where our own tasks were failing for the same reason ( so, since this time I made some changes in the role itself , I'll wait for your green light to update our collection on galaxy. |
Beta Was this translation helpful? Give feedback.
-
@TheGoldenPlatypus it's been included https://forum.ansible.com/t/default-date-2025-08-04-collection-inclusion-include-ravendb-ravendb-in-the-ansible-community-package/44150/12. |
Beta Was this translation helpful? Give feedback.
-
We’re excited to see our collection officially included in the Ansible community package. btw, will the collection be included in the next 11.x release? when is the next 11.x release planned? |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
An Ansible collection for deploying, configuring, and managing RavenDB clusters on Linux-based environments.
Ansible Galaxy: https://galaxy.ansible.com/ui/repo/published/ravendb/ravendb/
GitHub: https://github.com/ravendb/ansible-collection-ravendb
Issues Tracker: https://issues.hibernatingrhinos.com/issues
GitHub Handles: @TheGoldenPlatypus @gregolsky @garayx @ppekrol
Is the collection part of Automation Hub: No
We meet Collection Requirements: Yes
Beta Was this translation helpful? Give feedback.
All reactions