-
Notifications
You must be signed in to change notification settings - Fork 9
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
Save Job History on Flink #131
Conversation
updates: - [github.com/psf/black: 23.10.1 → 23.11.0](psf/black@23.10.1...23.11.0)
for more information, see https://pre-commit.ci
…ngeo-forge-runner into gcorradini/historyserver
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #131 +/- ##
===========================================
- Coverage 96.06% 74.50% -21.57%
===========================================
Files 14 15 +1
Lines 458 502 +44
===========================================
- Hits 440 374 -66
- Misses 18 128 +110 ☔ View full report in Codecov by Sentry. |
I didn't want to make the EFS mounting optional for our deployment but I see now I probably should in case folks have older clusters 😞 |
for more information, see https://pre-commit.ci
…ngeo-forge-runner into gcorradini/historyserver
looks like I need some tests too |
Drop explicit dependency on pangeo-forge-recipes
for more information, see https://pre-commit.ci
In meta_yaml recipes section, `dict_object`s should be a list element
Reimplement unique job names from #84
for more information, see https://pre-commit.ci
Move everything to pyproject.toml
Fix release workflow for pyproject.toml
[pre-commit.ci] pre-commit autoupdate
for more information, see https://pre-commit.ci
Upgrade Flink
Restore optional cache target
for more information, see https://pre-commit.ci
Mount EFS to Job Managers so they can archive jobs for historical status lookups
Addresses #122
Related PR: pangeo-forge/pangeo-forge-cloud-federation#6