-
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-2789] update data-platform-workflows #263
Conversation
Codecov Report
@@ Coverage Diff @@
## main #263 +/- ##
==========================================
- Coverage 78.98% 78.92% -0.07%
==========================================
Files 10 10
Lines 2151 2149 -2
Branches 349 349
==========================================
- Hits 1699 1696 -3
- Misses 381 383 +2
+ Partials 71 70 -1 |
3e80172
to
624774c
Compare
624774c
to
bf0e086
Compare
cdfdbf5
to
422c8d0
Compare
b054d89
to
49e8f80
Compare
ee6a2fe
to
df36e8b
Compare
This reverts commit c4f72d2.
strategy: | ||
fail-fast: false | ||
max-parallel: 6 |
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.
Do we still want this?
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.
I think it's not needed based on Alex comments some time ago and also because we should move to self hosted runners later (which should have more runners available as I heard recently).
@pytest.fixture(scope="module") | ||
async def cloud_configs(ops_test: OpsTest) -> None: |
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.
Moved to backups
renovate.json
Outdated
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.
Most custom rules are dropped for now, will try to reenable what we need at a later date.
tox.ini
Outdated
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.
Poetry and shellcheck need to be installed separately.
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! Thanks Dragomir!
strategy: | ||
fail-fast: false | ||
max-parallel: 6 |
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.
I think it's not needed based on Alex comments some time ago and also because we should move to self hosted runners later (which should have more runners available as I heard recently).
* Tweak workflows * Initial pass * Tweak conftest * Move renovate configs * Add wrapper * Copy from mysql-k8s * Linting * Partial merge * Saner marks * Tweak excludes * Bump dpw version * Bump deps * Tweak backup tests * Conditional free space * Revert "Conditional free space" This reverts commit c4f72d2. * Bump jira sync version * Bump lock * Remove schedule and call * Bump libs and actions
Update to latest workflows and refactor CI to more closely resemble MySQL's.