Update flowetl.Dockerfile

Flowminder / FlowKit

FlowKit: Flowminder CDR analytics toolkit

https://flowminder.github.io/FlowKit/

Mozilla Public License 2.0

86 stars 20 forks source link

Update flowetl.Dockerfile #6685

Closed Thingus closed 5 days ago

Thingus commented 5 days ago

Closes #6507

I have:

[ ] Formatted any Python files with black
[x] Brought the branch up to date with master
[x] Added any relevant Github labels
[ ] Added tests for any new additions
[ ] Added or updated any relevant documentation
[ ] Added an Architectural Decision Record (ADR), if appropriate
[ ] Added an MPLv2 License Header if appropriate
[x] Updated the Changelog

Description

Bumps Airflow to 2.9.2
unpins flowetl's Black due to a dependency conflict
Sets the shell in flowetl's makefile explicitly to Bash (source was failing when make upgrade was run on Ubuntu, as it was defaulting to sh)
Bumps the ci image for flowetl to ubuntu 2404:2024.5.1
Removes the --user flag from the flowetl Dockerfile

ellipsis-dev[bot] commented 5 days ago

Your free trial has expired. To keep using Ellipsis, sign up at https://app.ellipsis.dev for $20/seat/month or reach us at help@ellipsis.dev

cypress[bot] commented 5 days ago

Passing run #23077 ↗︎

Details:

Neater changelog
Project: FlowAuth	Commit: `3c98400969`
Status: Passed	Duration: 00:38 💡
Started: Jul 4, 2024 3:17 PM	Ended: Jul 4, 2024 3:17 PM

Review all test suite changes for PR #6685 ↗︎

Thingus commented 5 days ago

Re #6507 - do we need to install into the --user for our airflow?

codecov[bot] commented 5 days ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 92.29%. Comparing base (8156b52) to head (3c98400).

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #6685 +/- ## ========================================== - Coverage 92.30% 92.29% -0.01% ========================================== Files 268 268 Lines 10590 10590 Branches 856 856 ========================================== - Hits 9775 9774 -1 Misses 676 676 - Partials 139 140 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.