Skip to content

Replace "...examples" with docker-example-omero and add prod-playbooks#7

Merged
lubianat merged 1 commit intoome:masterfrom
pwalczysko:update-ansible-docker-section
Dec 18, 2025
Merged

Replace "...examples" with docker-example-omero and add prod-playbooks#7
lubianat merged 1 commit intoome:masterfrom
pwalczysko:update-ansible-docker-section

Conversation

@pwalczysko
Copy link
Copy Markdown
Member

@pwalczysko pwalczysko commented Dec 18, 2025

@lubianat @joshmoore @jburel @Rdornier

First of all, thanks @lubianat for the very nice dashboard here.

The present PR is suggesting to restructure the ansible-docker section, in lines with comment at ome/docker-example-omero#21 (comment) - in a nutshell:

There are quite a few repos in https://github.com/ome/omero-deployment-examples the maintenance of which is unclear and unallocated (and quite frankly, for most of them not happening for quite a while). This was discussed with @jburel and we agree that only one will be supported/maintained going ahead. I would suggest 1 for docker and 1 for ansible.

For docker: the docker-example-omero
For ansible: prod-playbooks == the main public ansible playbooks repo which is being actively maintained (the GHA is running and green there, and the playbook is also tested manually during OME Teams prod server deployments)

The present PR is trying to reflect this ^^^ suggestion.

@pwalczysko
Copy link
Copy Markdown
Member Author

Sorry, I wanted to ask @lubianat for a review, but somehow I do not see him on the list in the Reviewers. Maybe @joshmoore or @jburel could help please ?

Also, the tests are passing for me on my fork https://github.com/pwalczysko/status/actions, but did not start with opening of this PR.

@pwalczysko
Copy link
Copy Markdown
Member Author

I belive the tests are starting at 6 am every day as per

but only from master.

Would you be happy for me to merge my PR and see what happens ?

@joshmoore joshmoore requested a review from lubianat December 18, 2025 13:17
@joshmoore
Copy link
Copy Markdown
Member

Maybe @joshmoore or @jburel could help please ?

Had to assign this repo to the gerbi team. 👍

Copy link
Copy Markdown
Contributor

@lubianat lubianat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

it may be a good option to set some kind of PR preview here, but I'll tackle that in a different issue

@lubianat
Copy link
Copy Markdown
Contributor

I'll just merge it and run the workflow manually for now.

@lubianat lubianat merged commit ee67da4 into ome:master Dec 18, 2025
@pwalczysko pwalczysko deleted the update-ansible-docker-section branch December 18, 2025 18:46
@pwalczysko
Copy link
Copy Markdown
Member Author

Thank you @lubianat and @joshmoore

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants