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

Adjusted user creation in UBI 8 image #58

Merged
merged 1 commit into from
Apr 18, 2024
Merged

Conversation

bradfordcp
Copy link
Member

@bradfordcp bradfordcp commented Apr 12, 2024

Added chown and chmod flags to COPY commands
Added update step to UBI 8
Updated UBI 8 to point at Docker Hub container
Removed UBI 7

@burmanm
Copy link
Contributor

burmanm commented Apr 16, 2024

@bradfordcp You had something to check still last time we talked, is this fine now?

@bradfordcp
Copy link
Member Author

The latest version works in my testing. I'm a little iffy on setting the definitions directory to 777, but omitting that leads to weird errors when the config builder is scanning definitions. The USER setting in the Dockerfile isn't always being respected. Do you think this is an issue given these files are all templates?

Added chown and chmod flags to COPY commands
Added update step to UBI 8
Updated UBI 8 to point at Docker Hub container
Removed UBI 7 container
Consolidated user account COPY to single statement
@bradfordcp
Copy link
Member Author

Setting the execute bit on the files wasn't sitting right with me. I refactored the ubi builder image to set the bit appropriately on directories and then copied those definitions in. Tested with the latest k8ssandra-operator in Helm and the following modified K8ssandraCluster

apiVersion: k8ssandra.io/v1alpha1
kind: K8ssandraCluster
metadata:
    name: foo
spec:
    auth: false
    cassandra:
        serverType: dse
        serverVersion: 6.8.43
        initContainers:
            - name: server-config-init
              image: bradfordcpdstx/cass-config-builder:1.0.9-dev-ubi-5

Note the named repo is deleted

@burmanm burmanm merged commit 0893875 into master Apr 18, 2024
1 check 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.

2 participants