Skip to content

bucket notifications - need connect files also in endpoint for put bucket notif#1520

Merged
alphaprinz merged 1 commit intonoobaa:masterfrom
alphaprinz:notif_fix
Feb 2, 2025
Merged

bucket notifications - need connect files also in endpoint for put bucket notif#1520
alphaprinz merged 1 commit intonoobaa:masterfrom
alphaprinz:notif_fix

Conversation

@alphaprinz
Copy link
Contributor

Explain the changes

  1. Since PR bucket notifications - validate notifications on change (gh issue 8649) noobaa-core#8667 the endpoint validate the new notifications conf before applying the change to DB. This means we need the connect files also in endpoint.

Issues: Fixed #xxx / Gap #xxx

  1. put bucket notifications stopped working in containerized after above PR was merged.

Testing Instructions:

  1. Simple put bucket notification scenario.
  • Doc added/updated
  • Tests added

…cket notif

Signed-off-by: Amit Prinz Setter <alphaprinz@gmail.com>
@alphaprinz alphaprinz merged commit 7bf76c4 into noobaa:master Feb 2, 2025
14 checks passed
alphaprinz added a commit to alphaprinz/noobaa-operator that referenced this pull request Feb 2, 2025
…cket notif (noobaa#1520)

Signed-off-by: Amit Prinz Setter <alphaprinz@gmail.com>
nimrod-becker pushed a commit that referenced this pull request Feb 3, 2025
…cket notif (#1520) (#1522)

Signed-off-by: Amit Prinz Setter <alphaprinz@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants