stackabletech / airflow-operator

Stackable Operator for Apache Airflow
Other
21 stars 2 forks source link

Also add volume mounts to the git-sync container #454

Closed PaulienVa closed 2 months ago

PaulienVa commented 2 months ago

Description

When using kubernetes executors, volume mounts are not mounted to the gitsync initContainer. This causes a problem when a certificate is needed to access a git repository (also when using https:// instead of ssh). The webserver and the scheduler are able to sync the repositories, to avoid an inconsistent state of the operator, the executors should do to.

Definition of Done Checklist

# Author
- [x] 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
stackable-bot commented 2 months ago

CLA assistant check
All committers have signed the CLA.

PaulienVa commented 2 months ago

Due to https://github.com/orgs/stackabletech/discussions/49