feat: Integrate listener operator #784

New issue

Jump to bottom

Merged

labrenbe merged 42 commits into main from feat/integrate-listener-operator

Jul 4, 2025

+800 −486

Member

labrenbe commented Apr 25, 2025 •

edited by maltesander

Loading

Description

fixes: #688

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
- [x] Changes are OpenShift compatible
- [x] CRD changes approved
- [x] CRD documentation for all fields, following the [style guide](https://docs.stackable.tech/home/nightly/contributor/docs/style-guide).
- [x] Helm chart can be installed and deployed operator works
- [x] Integration tests passed (for non trivial changes)
- [x] Changes need to be "offline" compatible

# Reviewer
- [ ] Code contains useful comments
- [ ] Code contains useful logging statements
- [x] (Integration-)Test cases added
- [x] Documentation added or updated. Follows the [style guide](https://docs.stackable.tech/home/nightly/contributor/docs/style-guide).
- [x] Changelog updated
- [x] 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

labrenbe added 2 commits

April 25, 2025 15:00


          add support for listeners

60eb387


          add very basic integration test

93120b3

labrenbe self-assigned this

labrenbe changed the title ~~feat: Iintegrate listener operator~~ feat: Integrate listener operator

nightkr mentioned this pull request

NiFi v2 Behind Openshift Route #750

Open

labrenbe added 3 commits

May 16, 2025 16:41


          Merge remote-tracking branch 'origin/main' into feat/integrate-listen…

844d198

…er-operator


          add pr number to changelog

eb5136a


          add external-access test

c1aede3

labrenbe marked this pull request as ready for review

May 16, 2025 15:19

labrenbe requested a review from a team

May 19, 2025 12:34

labrenbe and others added 3 commits

May 21, 2025 15:30


          chore: Add NiFi 2.4.0 and remove 2.2.0 (#797)

46b3c14

add NiFi 2.4.0 and remove 2.2.0


          test: Add test for Apache Iceberg integration (#785)

d33ffab

* Clean up smoke test

* clean up smoke test part 2

* Add working test :)

* Move files

* Add and test HDFS functionality

* Kerbize HDFS and HMS

* Add Kerberos test

* Use nightly image

* linter

* Update Iceberg docs

* changelog

* Small bumps

* Update docs/modules/nifi/pages/usage_guide/writing-to-iceberg-tables.adoc

Co-authored-by: Nick <[email protected]>

---------

Co-authored-by: Nick <[email protected]>


          wip: update listener implementation based on implementation for super…

1b80397

…set operator

NickLarsenNZ reviewed

View reviewed changes

deploy/helm/nifi-operator/templates/roles.yaml Show resolved Hide resolved

labrenbe added 3 commits

June 3, 2025 10:13


          Merge remote-tracking branch 'origin/main' into feat/integrate-listen…

ed5f1dc

…er-operator


          use single listener with pvc per rolegroup

fcb0c4a


          update integration tests with crd change

b5202ff

maltesander self-requested a review

June 6, 2025 12:10

maltesander requested changes

View reviewed changes

CHANGELOG.md Outdated Show resolved Hide resolved

rust/operator-binary/src/crd/utils.rs Outdated Show resolved Hide resolved

rust/operator-binary/src/controller.rs Outdated Show resolved Hide resolved

rust/operator-binary/src/controller.rs Outdated Show resolved Hide resolved

rust/operator-binary/src/controller.rs Show resolved Hide resolved

rust/operator-binary/src/controller.rs Outdated Show resolved Hide resolved

tests/templates/kuttl/iceberg/50_nifi.yaml.j2 Outdated Show resolved Hide resolved

rust/operator-binary/src/crd/utils.rs Outdated Show resolved Hide resolved

rust/operator-binary/src/controller.rs Show resolved Hide resolved

labrenbe added 6 commits

June 10, 2025 10:41


          restore iceberg test file

a29e789


          fix listener class in iceberg test

a99adff


          Merge remote-tracking branch 'origin/main' into feat/integrate-listen…

95ae9f5

…er-operator


          expose https port in headless service

dd63d75


          update comment

afc55ac


          address feedback from review

bf88cc1

maltesander reviewed

View reviewed changes

rust/operator-binary/src/crd/mod.rs Outdated Show resolved Hide resolved

rust/operator-binary/src/controller.rs Outdated Show resolved Hide resolved

rust/operator-binary/src/controller.rs Outdated Show resolved Hide resolved

rust/operator-binary/src/controller.rs Show resolved Hide resolved

labrenbe added 4 commits

June 12, 2025 14:26


          remove unused error variants

7262d89


          create headless service name in function

e950a5a


          remove unused functions

41819bb


          move listenerClass to roleConfig

190ca21


          use new headless service name in integration tests

ef07f0c

maltesander reviewed

View reviewed changes

tests/test-definition.yaml Outdated Show resolved Hide resolved

rust/operator-binary/src/crd/mod.rs Outdated Show resolved Hide resolved

rust/operator-binary/src/controller.rs Outdated Show resolved Hide resolved

rust/operator-binary/src/controller.rs Outdated Show resolved Hide resolved

labrenbe added 3 commits

June 23, 2025 10:36


          move listener constants to listener module

b029440


          remove duplicate iceberg test

a4e3313


          remove hard-coded names

eb06291

maltesander reviewed

View reviewed changes

rust/operator-binary/src/listener.rs Outdated Show resolved Hide resolved

rust/operator-binary/src/controller.rs Show resolved Hide resolved

labrenbe added 2 commits

June 23, 2025 16:26


          fix integration tests

3c7850f


          remove hardcoded role name

79a1930

maltesander reviewed

View reviewed changes

rust/operator-binary/src/controller.rs Outdated Show resolved Hide resolved

rust/operator-binary/src/controller.rs Outdated Show resolved Hide resolved


          improve code quality

f38deac

adwk67 reviewed

View reviewed changes

rust/operator-binary/src/controller.rs Show resolved Hide resolved

labrenbe added 13 commits

June 23, 2025 18:27


          set rolegroup label on listener pvcs to none

bbbc4ad


          fix app version label on listener

c5c2264


          set rolegroup label on listener pvcs


          Merge remote-tracking branch 'origin/main' into feat/integrate-listen…

83027b8

…er-operator


          remove listener class from trino in iceberg test


          add note on custom ListenerClasses to docs

b8c87e3


          remove version argument in reporting task

074d138


          rename headless service

3f8d840


          fix iceberg test

c7ce9b6


          create separate headless services

8658f8b


          add missing file

b83120d


          fix iceberg test

bb49f43


          use listener scope for tls

05ada01

maltesander reviewed

View reviewed changes

docs/modules/nifi/pages/usage_guide/monitoring.adoc Show resolved Hide resolved

docs/modules/nifi/pages/usage_guide/monitoring.adoc Show resolved Hide resolved

maltesander approved these changes

View reviewed changes

Member

maltesander left a comment

LGTM!

labrenbe added this pull request to the merge queue

Merged via the queue into main with commit d4a101b

17 checks passed

labrenbe deleted the feat/integrate-listener-operator branch

July 4, 2025 09:08

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet