Skip to content
Snippets Groups Projects

[Run2_2017] Add a new Docker image for ServiceX

Merged Alexx Perloff requested to merge github/fork/aperloff/Run2_2017_Docker into Run2_2017

Created by: aperloff

Add the automatic creation of a new Docker image for ServiceX. This creation will take place after the base image for TreeMaker and simultaneous to the integration test. The reason this image is needed is to add some customization to the python packages and environment variables.

In doing this work, several updated were made to the GitHub actions workflow.

  1. Publishing is turned off for all users except 'TreeMaker'. While this isn't totally secure, it prevents innocent users from accidentally overwriting the images if they happen to have the DockerHub secrets.
  2. The integration-test and service-x workflows won't run if the user isn't 'TreeMaker'. As the image won't be updated unless the user is merging in the Run2_2017 branch after a PR and the users account can't publish the resulting images, it doesn't make sense to go through the effort for them to run a second , but unpublished, integration test.

Because of the changes mentioned above, I was unable to test to ServiceX workflow in it's entirety. I was able to test that if one uses the treemaker/treemaker:Runw_2017-latest image, that the .github/ServiceX/scripts/setup.sh works. The only part that is untested is the .github/ServiceX/scripts/setup-root.sh part.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Kevin Pedro
  • Kevin Pedro
  • Kevin Pedro
    Kevin Pedro @pedrok started a thread on commit f531602e
  • 1 #!/usr/bin/env bash
    2 #sudo mkdir -p /etc/grid-security
  • Kevin Pedro
    Kevin Pedro @pedrok started a thread on commit f531602e
  • 1 #!/usr/bin/env bash
    2 #sudo mkdir -p /etc/grid-security
    3 #sudo chown cmsuser /etc/grid-security
    4
    5
    6 while true; do
    7 #sudo cp /etc/grid-security-ro/x509up /etc/grid-security
  • Alexx Perloff @aperloff started a thread on commit f531602e
  • 1 #!/usr/bin/env bash
    2 #sudo mkdir -p /etc/grid-security
    • Author Contributor

      Created by: aperloff

      That is there on purpose. It will be needed eventually, but for this first test @mweinberg2718 and I are sidestepping anything having to do with sudo or grid proxies.

  • Alexx Perloff @aperloff started a thread on commit f531602e
  • 1 #!/usr/bin/env bash
    2 #sudo mkdir -p /etc/grid-security
    3 #sudo chown cmsuser /etc/grid-security
    4
    5
    6 while true; do
    7 #sudo cp /etc/grid-security-ro/x509up /etc/grid-security
    Please register or sign in to reply
    Loading