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

Container service updates #141

Merged
merged 5 commits into from
Oct 11, 2024
Merged

Container service updates #141

merged 5 commits into from
Oct 11, 2024

Conversation

tomdoel
Copy link
Collaborator

@tomdoel tomdoel commented Oct 10, 2024

Adds automatic pulling/updating of container service images as part of ansible run.
This is useful in helping to keep the container service up to date with the latest images.

I have also added a REST call to enable the AutoRun pipeline if a flag is set. The AutoRun pipeline is currently needed to send session archive notification emails, but can be disabled (default) if these are not required.

@tomdoel tomdoel requested a review from a team October 10, 2024 15:54
@tomdoel tomdoel requested review from a team and removed request for a team October 10, 2024 16:06
Copy link
Contributor

@drmatthews drmatthews left a comment

Choose a reason for hiding this comment

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

Small typo, otherwise looks good.

roles/xnat/tasks/configure.yml Outdated Show resolved Hide resolved
Co-authored-by: Daniel Matthews <[email protected]>
@tomdoel tomdoel requested a review from drmatthews October 11, 2024 09:22
Copy link
Contributor

@drmatthews drmatthews left a comment

Choose a reason for hiding this comment

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

👍

@tomdoel tomdoel merged commit 5596400 into main Oct 11, 2024
22 checks passed
@tomdoel tomdoel deleted the container-service-updates branch October 11, 2024 09:23
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.

2 participants