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

Add Podman Deployment #311

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

Add Podman Deployment #311

wants to merge 12 commits into from

Conversation

muhwagwa
Copy link
Collaborator

@muhwagwa muhwagwa commented Jun 7, 2024

Add Podman SCIM deployment under beta repo

@muhwagwa muhwagwa self-assigned this Jun 7, 2024
@muhwagwa muhwagwa marked this pull request as draft June 7, 2024 19:49
@muhwagwa muhwagwa marked this pull request as ready for review June 10, 2024 13:27
cd scim-examples/
git pull
cd beta/podman/
podman-compose up --build -d
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@ag-rdoucette Would this have to change to redeploying the bridge using deploy.sh?

@renatarocha-1pass
Copy link

Do we have any updates on this one?

@muhwagwa
Copy link
Collaborator Author

muhwagwa commented Oct 9, 2024

Do we have any updates on this one?

@ag-rdoucette will be adding in updates by eow. Once that's done and tested, we'll close this PR.

@Macleykun
Copy link

Hi, was curious how things are going related to this PR!

@ag-rdoucette
Copy link
Collaborator

@Macleykun ! Sorry for the delays in this; I've been fairly wrapped up on some other projects.

The only item really missing is the ReadMe, and I'll try to find some time next week to finish that. I have already used this deployment with a few clients and on a couple of my VMs, and it appears to be working just fine.

@Macleykun
Copy link

@Macleykun ! Sorry for the delays in this; I've been fairly wrapped up on some other projects.

The only item really missing is the ReadMe, and I'll try to find some time next week to finish that. I have already used this deployment with a few clients and on a couple of my VMs, and it appears to be working just fine.

no need to sorry! Glad to hear the setup is working well so far! When my team needs to (re)setup the scim bridge, we'll use this PR/merged PR to set it up more securely!

Copy link
Member

@plttn plttn left a comment

Choose a reason for hiding this comment

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

Are we missing deploy.sh currently?

We may also want to add how to properly service-ize this deployment to ensure it stays up when the host is restarted.

stretch goal for a future PR: use my newfound podman skills to make this use quadlets, which are now preferred over podman-compose

@ag-rdoucette
Copy link
Collaborator

ag-rdoucette commented Dec 11, 2024

@plttn Removed the deploy.sh and instead putting instructions on how to create the secrets before using podman compose commands to stand up the bridge.

Commands with podman are different in that we must call on the env file specifically to get this working.

I was also looking at quadlets and services, but there are also differences between Fedora and Redhat and I'm working out the best path forward on what to include in the ReadMe.

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.

5 participants