-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[DPE-2904] Switch to self hosted runners #285
Conversation
This reverts commit 24c9f8c.
@@ -57,24 +57,24 @@ coverage = {extras = ["toml"], version = "^7.3.2"} | |||
pytest = "^7.4.0" | |||
pytest-asyncio = "^0.21.1" | |||
jsonschema = "^4.19.1" | |||
psycopg2 = {version = "^2.9.9", extras = ["binary"]} | |||
psycopg2-binary = "^2.9.9" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No libpq on the runner.
0f454f6
to
c62d355
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- LGTM.
- Please update CONTRIBUTION.md guilde to represent new way to run the test(s):
tox run -e integration -- 'tests/integration/ha_tests/test_restore_cluster.py' --group='1' -m 'not unstable' --model test
bf0f0ef
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but tox run -e integration # integration tests
is still wrong in CONTRIBUTION.md, isn't it?
I tried and |
.github/workflows/ci.yaml
Outdated
- lint | ||
- unit-test | ||
uses: canonical/data-platform-workflows/.github/workflows/integration_test_charm.yaml@test-large | ||
- build | ||
uses: canonical/data-platform-workflows/.github/workflows/[email protected] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fyi this is gh-hosted, not self-hosted
3b20d75
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's the party begin!
* DO NOT MERGE test self-hsoted runners * Remove groups * Add group markers * fix pythonpath * Switch to relative imports * Switch to binary psycopg2 for test deps * Conditionally skip tests * Bump timeouts * Skipping fixtures and bump in timeouts * Juju matrix * Revert "Juju matrix" This reverts commit 24c9f8c. * Juju2 job * Revert backup changes * Bump timeouts * Switch to dpw 6 * More timeout bumps * Increase timeouts * Bump dpw * Depend on unit tests * Increase timeouts * Skip additional backup test when secrets are not accessible * Even more timouts * Retry getting password if no primary was found * Timeouts * Bump dpw * Try to preenable plpython * Add markers and cleanup tox ini * Marks * Switch to mainline workflow * Bump actions * Matrix integration * Update contributing * Use the right workflow * Bump action --------- Co-authored-by: Carl Csaposs <[email protected]>
Switch to Canonical hosted runners.