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

Update for pass-auth consolidation #16

Merged
merged 2 commits into from
Apr 22, 2024
Merged

Update for pass-auth consolidation #16

merged 2 commits into from
Apr 22, 2024

Conversation

markpatton
Copy link
Contributor

@markpatton markpatton commented Mar 26, 2024

This pr updates the acceptance testing for the auth consolidation work.

In order to test you will need updated pass-core, pass-ui, and idp images. The code is in the 903-auth branch in all those repos.

Build the pass-core image:

git clone https://github.com/eclipse-pass/pass-core/
git checkout 903-auth
cd pass-core
mvn clean verify

Build the idp image:

git clone https://github.com/eclipse-pass/pass-docker/
git checkout 903-auth
docker compose -f docker-compose.yml -f eclipse-pass.local.yml build idp

Build the pass-ui image using the 903-auth branch.

git clone https://github.com/eclipse-pass/pass-ui/
git checkout 903-auth
./build.sh ../pass-docker/.env
docker image tag ghcr.io/eclipse-pass/pass-ui:latest ghcr.io/eclipse-pass/pass-ui:1.6.0-SNAPSHOT

Then in pass-docker in the 903-auth branch:

docker compose -f docker-compose.yml -f eclipse-pass.local.yml up -d

Test the environment by hitting http://localhost:8080/app/.

Then run the acceptance tests like normal.

Copy link
Contributor

@jaredgalanis jaredgalanis left a comment

Choose a reason for hiding this comment

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

Seems good 👍

@markpatton markpatton merged commit 92edcf7 into main Apr 22, 2024
1 of 2 checks passed
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.

Consolidate pass-auth and pass-docker reverse proxy functionality into pass-core
2 participants