Improve waits in integration tests #1322
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
❌ Deploy Preview for galaxyng failed.
|
|
/retest |
awcrosby
force-pushed
the
fix/integration_waits
branch
3 times, most recently
from
5e7f7c8
to
dc138cc
Compare
|
/retest |
awcrosby
force-pushed
the
fix/integration_waits
branch
from
dc138cc
to
643cfaf
Compare
|
/retest |
2 similar comments
|
/retest |
|
/retest |
awcrosby
force-pushed
the
fix/integration_waits
branch
from
643cfaf
to
edc618a
Compare
|
/retest |
awcrosby
force-pushed
the
fix/integration_waits
branch
2 times, most recently
from
7ae7742
to
0cc97d7
Compare
|
/retest |
awcrosby
force-pushed
the
fix/integration_waits
branch
from
0cc97d7
to
2977173
Compare
awcrosby
force-pushed
the
fix/integration_waits
branch
from
2977173
to
5e69fea
Compare
awcrosby
force-pushed
the
fix/integration_waits
branch
from
5e69fea
to
63e4dd7
Compare
awcrosby
changed the title
jctanner
approved these changes
Jun 22, 2022
adityamulik
pushed a commit
to adityamulik/galaxy_ng
that referenced
this pull request
Jul 7, 2022
* Use wait_for_task util where possible * Add a wait_for_url util to better poll 404s No-Issue
adityamulik
added a commit
to adityamulik/galaxy_ng
that referenced
this pull request
Jul 7, 2022
- pre_translation.sh - post_translation.sh Ref: https://galaxy.ansible.com/ansible/memsource Turn off collection signing in pr_check.sh (ansible#1324) No-Issue Update CRC signature defaults to become python boolean (ansible#1321) No-Issue Poll the move task(s) during integration (ansible#1325) * Poll the move task(s) during integration * Poll tasks in explicit order. No-Issue Signed-off-by: James Tanner <[email protected]> Improve waits in integration tests (ansible#1322) * Use wait_for_task util where possible * Add a wait_for_url util to better poll 404s No-Issue set signing keyring for staging and published repos (ansible#1319) Issue: AAH-1712 Proposal: provide the pulp apis as part of galaxy's supported API surface (ansible#1288) * Provide the pulp apis as part of galaxy's supported API surface Issue: AAH-1681 [bugfix] Fix signing feature flags for UI (ansible#1318) Issue: AAH-1690 Add script for updating packages across branches. (ansible#1296) No-Issue Signed-off-by: James Tanner <[email protected]> dev standalone-keycloak: use keycloak/keycloak:legacy, not latest (ansible#1323) when ansible#889 was merged, the current keycloak version was 15.0.2, since 17.0.0, docker run keycloak/keycloak just shows help instead of running the server and there's a *-legacy version which still runs the server Updating to latest legacy, tested with 18.0.1-legacy (this seems related to keycloak/keycloak@30d2dcb and making https://github.com/keycloak/keycloak-containers legacy) No-Issue [feature] Add LDAP integration (ansible#1327) Issue: AAH-1593 Co-authored-by: Chris Hambridge <[email protected]> Add user profiles to integration tests (ansible#1333) * Define allowed user profiles for ansible config fixture * Update the setup for test users * Use new org_admin user instead of context manager workaround * Ensure user is added to RH identity group even if user already created * Skip token auth for ephemeral Issue: AAH-1449 Edit integration test users to specify non-admins (ansible#1334) * Remove unused user profiles * Use the basic_user profile where possible * Explicitly use admin user where needed * Define test data in one place for all methods of calling tests * Remove an xfail to confirm test fails as expected No-Issue Update filter for indexing EEs from catalog.redhat.com (ansible#1346) Issue: AAH-1737 Add diagnose messages to the feature-flags endpoint (ansible#1332) Issue: AAH-1738 Improve integration tests and test data (ansible#1342) * Replace data fixture call with setup_test_data.py * Remove unused data setup scripts * Match python conditional to how bash checks for HUB_LOCAL * Stop skipping test for use_distribution, remove unneeded profiles * Unskip test_auth tests for standalone mode * Add integration test README No-Issue
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What is this PR doing:
Check if we need more and/or smarter waits for a potentially slower
ephemeral environment.
No-Issue