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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants