Skip to content

Add Airflow 2.8.4 and 2.9.2#461

Merged
maltesander merged 6 commits intomainfrom
chore/bump-versions
Jul 16, 2024
Merged

Add Airflow 2.8.4 and 2.9.2#461
maltesander merged 6 commits intomainfrom
chore/bump-versions

Conversation

@maltesander
Copy link
Member

@maltesander maltesander commented Jul 8, 2024

Description

  • added 2.8.4 and 2.9.2
  • removed 2.7.2 and 2.7.3

Definition of Done Checklist

  • Not all of these items are applicable to all PRs, the author should update this template to only leave the boxes in that are relevant
  • Please make sure all these things are done and tick the boxes
# Author
- [ ] Changes are OpenShift compatible
- [ ] CRD changes approved
- [ ] CRD documentation for all fields, following the [style guide](https://docs.stackable.tech/home/nightly/contributor/docs-style-guide).
- [ ] Helm chart can be installed and deployed operator works
- [ ] Integration tests passed (for non trivial changes)
- [ ] Changes need to be "offline" compatible

# Reviewer
- [ ] Code contains useful comments
- [ ] Code contains useful logging statements
- [ ] (Integration-)Test cases added
- [ ] Documentation added or updated. Follows the [style guide](https://docs.stackable.tech/home/nightly/contributor/docs-style-guide).
- [ ] Changelog updated
- [ ] Cargo.toml only contains references to git tags (not specific commits or branches)

# Acceptance
- [ ] Feature Tracker has been updated
- [ ] Proper release label has been added
- [ ] [Roadmap](https://github.com/orgs/stackabletech/projects/25/views/1) has been updated

@maltesander maltesander requested a review from a team July 8, 2024 14:13
@maltesander maltesander self-assigned this Jul 8, 2024
@maltesander maltesander mentioned this pull request Jul 8, 2024
Closed
@maltesander
Copy link
Member Author

maltesander commented Jul 11, 2024

test https://ci.stackable.tech/view/02%20Operator%20Tests%20(custom)/job/airflow-operator-it-custom/140/

@dervoeti dervoeti self-requested a review July 15, 2024 10:46
dervoeti
dervoeti reviewed Jul 15, 2024
View reviewed changes
Copy link
Member

@dervoeti dervoeti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, but I think we can remove these lines as well as they seem to be a workaround for 2.7.2 (which has now been removed). I ran the LDAP tests locally for 2.9.2 with the lines removed and they were successful.

@maltesander maltesander requested a review from dervoeti July 16, 2024 07:59
dervoeti
dervoeti approved these changes Jul 16, 2024
View reviewed changes
Copy link
Member

@dervoeti dervoeti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@maltesander maltesander added this pull request to the merge queue Jul 16, 2024
Merged via the queue into main with commit 0665fd8 Jul 16, 2024
@maltesander maltesander deleted the chore/bump-versions branch July 16, 2024 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dervoeti dervoeti dervoeti approved these changes

Assignees

@maltesander maltesander

Labels

release/24.7.0

Projects

Stackable Engineering
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@maltesander @dervoeti @lfrancke