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

[Open OnDemand] Remove task for ood_auth_map.regex permisisons #1068

Merged
merged 4 commits into from Dec 6, 2021

Conversation

ajdecon
Copy link
Collaborator

@ajdecon ajdecon commented Dec 6, 2021

Per the OOD release notes, the file /opt/ood/ood_auth_map/bin/ood_auth_map.regex is no longer being provided by default by OOD.

Including this task can lead to errors when setting up new OOD instances.

So let's remove it!

Test plan

Successfully execute open-ondemand.yml playbook

dholt and others added 4 commits September 15, 2021 12:35
Per the [OOD release
notes](https://osc.github.io/ood-documentation/latest/release-notes/v2.0-release-notes.html#no-longer-providing-ood-auth-map-regex),
the file `/opt/ood/ood_auth_map/bin/ood_auth_map.regex` is no longer
being provided by default by OOD.

Including this task can lead to errors when setting up new OOD
instances.

So let's remove it!

Test plan
---------

Successfully execute open-ondemand.yml playbook
@ajdecon ajdecon requested a review from dholt December 6, 2021 19:29
@ajdecon ajdecon mentioned this pull request Dec 6, 2021
@dholt
Copy link
Contributor

dholt commented Dec 6, 2021

Do you mind committing this against the 21.12 release branch? I already forked from master. Otherwise looks good.

@ajdecon ajdecon changed the base branch from master to release-21.12 December 6, 2021 22:55
@dholt dholt merged commit dbfad96 into NVIDIA:release-21.12 Dec 6, 2021
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.

None yet

2 participants