Jsonata Event transform e2e tests #8499
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Pierangelo Di Pilato <[email protected]>
Signed-off-by: Pierangelo Di Pilato <[email protected]>
|
Skipping CI for Draft Pull Request. |
Signed-off-by: Pierangelo Di Pilato <[email protected]>
f60a678 to
3e77fec
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8499 +/- ##
==========================================
+ Coverage 62.62% 62.81% +0.18%
==========================================
Files 396 396
Lines 24231 24231
==========================================
+ Hits 15175 15221 +46
+ Misses 8247 8201 -46
Partials 809 809 ☔ View full report in Codecov by Sentry. |
|
/lgtm |
Signed-off-by: Pierangelo Di Pilato <[email protected]>
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: matzew, pierDipi The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/test unit-tests |
2 similar comments
|
/test unit-tests |
|
/test unit-tests |
|
/retest |
Signed-off-by: Pierangelo Di Pilato <[email protected]>
a6aaf16 to
0e77d7d
Compare
|
/test unit-tests |
1 similar comment
|
/test unit-tests |
|
Sorry @matzew, I picked the wrong image SHA earlier, now it works |
|
/test unit-tests |
|
/lgtm |
matzew
pushed a commit
to matzew/eventing
that referenced
this pull request
Mar 18, 2025
* E2E tests Signed-off-by: Pierangelo Di Pilato <[email protected]> * EventTransform Jsonata e2e tests Signed-off-by: Pierangelo Di Pilato <[email protected]> * Use eventing-integrations images and fix tests Signed-off-by: Pierangelo Di Pilato <[email protected]> * Verify response status code received by source Signed-off-by: Pierangelo Di Pilato <[email protected]> * Update JSONata transformations Signed-off-by: Pierangelo Di Pilato <[email protected]> --------- Signed-off-by: Pierangelo Di Pilato <[email protected]>
matzew
pushed a commit
to matzew/eventing
that referenced
this pull request
Mar 19, 2025
* E2E tests Signed-off-by: Pierangelo Di Pilato <[email protected]> * EventTransform Jsonata e2e tests Signed-off-by: Pierangelo Di Pilato <[email protected]> * Use eventing-integrations images and fix tests Signed-off-by: Pierangelo Di Pilato <[email protected]> * Verify response status code received by source Signed-off-by: Pierangelo Di Pilato <[email protected]> * Update JSONata transformations Signed-off-by: Pierangelo Di Pilato <[email protected]> --------- Signed-off-by: Pierangelo Di Pilato <[email protected]>
openshift-merge-bot bot
pushed a commit
to openshift-knative/eventing
that referenced
this pull request
Mar 21, 2025
* Partial implementation of certmanager for integration sink (knative#8481) * Adding cert-mananger bits Basics for Cert-manager Signed-off-by: Matthias Wessendorf <[email protected]> * Adding create cert manifest function Signed-off-by: Matthias Wessendorf <[email protected]> * Use the make cert Signed-off-by: Matthias Wessendorf <[email protected]> * Reconcile Certificate manifest and add RBAC Signed-off-by: Matthias Wessendorf <[email protected]> * Add secret filter to controller Signed-off-by: Matthias Wessendorf <[email protected]> * Look up secrets/certs for sink Signed-off-by: Matthias Wessendorf <[email protected]> * Remove old/wrong Certificate manifest and reference Signed-off-by: Matthias Wessendorf <[email protected]> * Fix controller compile Signed-off-by: Matthias Wessendorf <[email protected]> * Formatting Signed-off-by: Matthias Wessendorf <[email protected]> * change the reconcile of cert manifest, and run it only w/ some form of TLS Signed-off-by: Matthias Wessendorf <[email protected]> * using filtered informer Signed-off-by: Matthias Wessendorf <[email protected]> * Adding Rekt test for IntegrationSink TLS support Signed-off-by: Matthias Wessendorf <[email protected]> * adding a bit of https port for deployment/service of the IntegrationSink Signed-off-by: Matthias Wessendorf <[email protected]> * Mounting secrets and setting quarkus env vars for TLS support Signed-off-by: Matthias Wessendorf <[email protected]> * Adding flags for different TLS levels resulting in diffeerent quarkus env vars Signed-off-by: Matthias Wessendorf <[email protected]> * Remove bad comment Signed-off-by: Matthias Wessendorf <[email protected]> * Adding label Signed-off-by: Matthias Wessendorf <[email protected]> * Update codegen Signed-off-by: Matthias Wessendorf <[email protected]> * Disable tmp. the cert manager feature Signed-off-by: Matthias Wessendorf <[email protected]> * Fixing header/year Signed-off-by: Matthias Wessendorf <[email protected]> * More test tweaking Signed-off-by: Matthias Wessendorf <[email protected]> * Remove unused and comment out the hooks for now Signed-off-by: Matthias Wessendorf <[email protected]> --------- Signed-off-by: Matthias Wessendorf <[email protected]> * Run make generate-release Signed-off-by: Matthias Wessendorf <[email protected]> * Remove unused code (knative#8485) Signed-off-by: Matthias Wessendorf <[email protected]> * IntegrationSink prefactors (knative#8486) Little refactors on IntegrationSink Signed-off-by: Matthias Wessendorf <[email protected]> * Add initial iteration of eventing-integration resources (knative#8493) * Generic code for Certificates (knative#8489) Make Certifacte factory more generic Signed-off-by: Matthias Wessendorf <[email protected]> * Jsonata Event transform e2e tests (knative#8499) * E2E tests Signed-off-by: Pierangelo Di Pilato <[email protected]> * EventTransform Jsonata e2e tests Signed-off-by: Pierangelo Di Pilato <[email protected]> * Use eventing-integrations images and fix tests Signed-off-by: Pierangelo Di Pilato <[email protected]> * Verify response status code received by source Signed-off-by: Pierangelo Di Pilato <[email protected]> * Update JSONata transformations Signed-off-by: Pierangelo Di Pilato <[email protected]> --------- Signed-off-by: Pierangelo Di Pilato <[email protected]> * Inject integration images from ConfigMap (knative#8500) Signed-off-by: Pierangelo Di Pilato <[email protected]> * Run make generate-release Signed-off-by: Matthias Wessendorf <[email protected]> * Update to the correct file (knative#8507) Signed-off-by: Matthias Wessendorf <[email protected]> * EventTransform: Inject tracing configurations (knative#8479) Signed-off-by: Pierangelo Di Pilato <[email protected]> * EventTransform: add reconciler unit tests (knative#8513) * EventTransform: add reconciler unit tests Signed-off-by: Pierangelo Di Pilato <[email protected]> * Update pkg/reconciler/testing/v1alpha1/eventtransform.go Co-authored-by: Marek Schmidt <[email protected]> --------- Signed-off-by: Pierangelo Di Pilato <[email protected]> Co-authored-by: Marek Schmidt <[email protected]> * Enable Cert Manager (knative#8509) * Re-enable Cert Mgr Signed-off-by: Matthias Wessendorf <[email protected]> * Manual informer Signed-off-by: Matthias Wessendorf <[email protected]> * Update main Signed-off-by: Matthias Wessendorf <[email protected]> * update controller Signed-off-by: Matthias Wessendorf <[email protected]> * Adding label on cert Signed-off-by: Matthias Wessendorf <[email protected]> * add enqueue Signed-off-by: Matthias Wessendorf <[email protected]> * Create Cert Informer, on demand, and cancle when flag is disabled Signed-off-by: Matthias Wessendorf <[email protected]> * Update unit tests that broke during rrefactors Signed-off-by: Matthias Wessendorf <[email protected]> --------- Signed-off-by: Matthias Wessendorf <[email protected]> * Event Transform JSONata TLS (knative#8515) * Event Transform JSONata TLS Signed-off-by: Pierangelo Di Pilato <[email protected]> * HA and security context Signed-off-by: Pierangelo Di Pilato <[email protected]> * Reconcile Certificate Signed-off-by: Pierangelo Di Pilato <[email protected]> * Additional unit tests cases Signed-off-by: Pierangelo Di Pilato <[email protected]> * Fix tests Signed-off-by: Pierangelo Di Pilato <[email protected]> * EventTransform trust bundle propagation and tests Signed-off-by: Pierangelo Di Pilato <[email protected]> * Fix tests and address comments Signed-off-by: Pierangelo Di Pilato <[email protected]> --------- Signed-off-by: Pierangelo Di Pilato <[email protected]> * Run make generate-release Signed-off-by: Matthias Wessendorf <[email protected]> * Add dynamic cert-manager certificates informer (knative#8517) * Add dynamic informer, part I Signed-off-by: Matthias Wessendorf <[email protected]> * Add Dynamic certificates informer for EventTransform Signed-off-by: Pierangelo Di Pilato <[email protected]> * Add Dynamic certificates Informer to IntegrationSink Signed-off-by: Pierangelo Di Pilato <[email protected]> * Fix error message Signed-off-by: Pierangelo Di Pilato <[email protected]> --------- Signed-off-by: Matthias Wessendorf <[email protected]> Signed-off-by: Pierangelo Di Pilato <[email protected]> Co-authored-by: Pierangelo Di Pilato <[email protected]> * Improve CM Cert reconcile and deletion if feature is turned off (knative#8519) refactor CM Cert reconciler and take care of delete if feature is disabled Signed-off-by: Matthias Wessendorf <[email protected]> * Remove transformations and integrations ConfigMaps (#1201) We don't use the ConfigMaps to inject the values, the operator will replace the value to the images directly. Signed-off-by: Pierangelo Di Pilato <[email protected]> Co-authored-by: Pierangelo Di Pilato <[email protected]> * Run make generate-release Signed-off-by: Matthias Wessendorf <[email protected]> * Add configmaps/finalizers permissions Signed-off-by: Pierangelo Di Pilato <[email protected]> * Add update finalizers permissions for EventTransform reconciled resources Signed-off-by: Pierangelo Di Pilato <[email protected]> * Volume mounts use direct Signed-off-by: Matthias Wessendorf <[email protected]> * Fix EventTransform with TLS Sink test Signed-off-by: Matthias Wessendorf <[email protected]> --------- Signed-off-by: Matthias Wessendorf <[email protected]> Signed-off-by: Pierangelo Di Pilato <[email protected]> Co-authored-by: David Simansky <[email protected]> Co-authored-by: Pierangelo Di Pilato <[email protected]> Co-authored-by: Marek Schmidt <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Waiting for knative-extensions/eventing-integrations#72 to be merged and built