Skip to content

Conversation

@jerabekjiri
Copy link
Contributor

@jerabekjiri jerabekjiri commented Jan 12, 2026

SUMMARY

Fixes: https://issues.redhat.com/browse/AAP-61962

ISSUE TYPE
  • New or Enhanced Feature
COMPONENT NAME
  • API
ADDITIONAL INFORMATION
  • Requires setting secrets.QUAY_TOKEN and secrets.QUAY_USER (though QUAY_USER may not need to be a secret)
  • My idea is to run this in the AAP repository only, which should have all branches (devel and stable-* branches)
@jerabekjiri
Copy link
Contributor Author

@AlanCoding updated, now it re-use docker-compose-buildx cmd for the tower_devel by overwriting DEV_DOCKER_TAG_BASE=quay.io/aap". It'd be good to test this works locally:

  1. docker login quay.io
  2. make docker-compose-buildx
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
4 Security Hotspots

See analysis details on SonarQube Cloud

Copy link
Member

@AlanCoding AlanCoding left a comment

Choose a reason for hiding this comment

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

I think you need to add a condition similar to this.

      - name: Skipping build of awx image for non-awx repository
        run: |
          echo "Skipping build of awx image for non-awx repository"
          exit 0
        if: matrix.build-targets.image-name == 'awx' && !endsWith(github.repository, '/awx')

This already exists in some other cases.

Why? Because https://quay.io/repository/aap/tower_devel is the objective here. But there is no https://quay.io/repository/aap/awx_devel, nor do we have a need for that, nor do we want that. In those cases, you want a public image, and use the ghcr awx_devel image that's already there.

To put it into words how this would be different - in your case you want to skip building the image for any non-tower repository. And maybe even more specific than that, probably needs to be specific to that 1 github org & repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

3 participants