Skip to content

two tests-scan adjustments - #9264

Open
allisonkarlitskaya wants to merge 2 commits into
mainfrom
queue-fixes
Open

two tests-scan adjustments#9264
allisonkarlitskaya wants to merge 2 commits into
mainfrom
queue-fixes

Conversation

@allisonkarlitskaya

Copy link
Copy Markdown
Member

...helping us get ready for AWS runners.

This is already the default in job-runner and it should be more than
enough.  Making it explicit helps us know how long to issue transient
AWS S3 tokens for.
Send 'rhel' jobs also to the public queue so that they run on AWS (which
only checks the public queue because it's outside of the VPN).  The one
exception to this is subscription-manager which has an open issue[1]
that needs fixing before it can run in AWS.

[1] github.com/cockpit-project/subscription-manager-cockpit/issues/231
Comment thread tests-scan
},
"command_subject": command_subject,
"slug": slug,
"timeout": 120,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Kinda long, but lets keep the old timeout alive for now.

Comment thread tests-scan
context = entry['job']['context']
queue = 'rhel' if is_internal_context(context) else 'public'
repo = entry['job']['repo']
is_subman = 'subscription-manager' in context or 'subscription-manager' in repo

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We need to discuss this, there is a pull request to get rid of this.

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