-
-
Notifications
You must be signed in to change notification settings - Fork 3
chore: Listener class at webserver role level only #645
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
Conversation
maltesander
left a comment
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.
There are a couple of .webservers.config.listenerClass references left, e.g. in doc examples.
The mount-dags-gitsync fail for me (metrics check does not work).
Changes in d83dfa0 |
maltesander
left a comment
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 if tests work!
🟢 Jenkins |

Description
🟢 Tests (running mount-configmap and mount-gitsync to test the commonly-used scripts, as well as test-specific ones): https://testing.stackable.tech/view/02%20Operator%20Tests%20(custom)/job/airflow-operator-it-custom/33/
Definition of Done Checklist
Author
Reviewer
Acceptance
type/deprecationlabel & add to the deprecation scheduletype/experimentallabel & add to the experimental features tracker