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.

Sign up for GitHub

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

Jump to bottom

[TEST] Dockerfile fix usergroup #238

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

[TEST] Dockerfile fix usergroup #238

wants to merge 28 commits into from

Conversation

amartinezfayo
Copy link
Owner

Pull Request check list

  • Commit conforms to CONTRIBUTING.md?
  • Proper tests/regressions included?
  • Documentation updated?

Affected functionality

Description of change

Which issue this PR fixes

amartinezfayo and others added 28 commits March 11, 2024 16:52
@amartinezfayo
Fix Dockerfile to have the proper permissions in directories
3c5c3f4 
Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Use user 1000 instead of 1001 in integration tests
6a53fdb 
Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Have WORKDIR after COPY
68be7e3 
Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Have WORKDIR after COPY
b8b7df0 
Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Fix permission in generated key file for tests
d75e62a 
Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Fix permission in shared folders for agent socket
061d418 
Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Prepare common directories used by tests, using proper permissions
92c9dcf 
Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Revert changes of uid
3b9cedc 
Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Revert changes of uid
2508f01 
Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Use uid 1000 instead of 0
33d83d8 
Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Dockerfile changes
d605288 
Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Avoid mounting volumes that are not needed
ba113fe 
Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Use uid 1000 instead of 1001 in svidstore integration test
3724d67 
Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Update permissions in shared directores for nested rotation integrati…
6ba64d0 
…on test

Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Fix root-agent Docker compose definition in the nested rotation integ…
3fd06ed 
…ration test to be able to access the Docker daemon socket

Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Fix k8s integration test
62b12d4 
Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Fix Delegated Identity API integration test
1038bcc 
Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Fix node attestation integration test
7745e96 
Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Fix Ghostunnel-Federation integration test
040b985 
Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Fix envoy-sds-v3 integration test
6571b6a 
Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Fix oidc-discovery-provider integration test
f3e1e71 
Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Fix envoy-sds-v3-spiffe-auth integration test
d112fcb 
Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Update Dockerfile
12b4ef6 
Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Update Dockerfile
914cc93 
Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Do not run as root when not needed. Remove WORKDIR in Dockerfile
8cf4578 
Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Fix integration tests
1b94174 
Signed-off-by: Agustín Martínez Fayó <[email protected]>
@amartinezfayo
Add comment about securityContext needed for hostPath volume
8dd99c7 
Signed-off-by: Agustín Martínez Fayó <[email protected]>
@azdagron
Merge branch 'main' into dockerfile-fix-usergroup
8f388e6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@amartinezfayo @azdagron