Skip to content

Adapt EC for eventing manager #17852

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

Merged
merged 5 commits into from
Jul 19, 2023

Conversation

muralov
Copy link
Contributor

@muralov muralov commented Jul 18, 2023

Description
Adapt Eventing Controller to be able to be used by Eventing Manager

Changes proposed in this pull request:

  • Make set EPP owner EC setting with flag so that we can set Eventing CR as owner
  • Add some methods used by EM to update NATS and publisher config

Related issue(s)
issue

@muralov muralov requested a review from a team as a code owner July 18, 2023 14:03
@muralov muralov requested a review from friedrichwilken July 18, 2023 14:03
@muralov muralov added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jul 18, 2023
@netlify
Copy link

netlify bot commented Jul 18, 2023

Deploy Preview for kyma-project-docs-preview ready!

Name Link
🔨 Latest commit 5b58f33
🔍 Latest deploy log https://app.netlify.com/sites/kyma-project-docs-preview/deploys/64b7c0a1220d6e000829bd32
😎 Deploy Preview https://deploy-preview-17852--kyma-project-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@kyma-bot kyma-bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jul 18, 2023
@muralov muralov force-pushed the adapt-for-eventing-manager branch 2 times, most recently from 66068c8 to 8529ebc Compare July 18, 2023 15:30
@muralov muralov force-pushed the adapt-for-eventing-manager branch from 8529ebc to 6103b96 Compare July 19, 2023 07:58
@muralov muralov changed the title Adapt for eventing manager Adapt EC for eventing manager Jul 19, 2023
@muralov muralov linked an issue Jul 19, 2023 that may be closed by this pull request
4 tasks
@kyma-bot kyma-bot added the lgtm Looks good to me! label Jul 19, 2023
@muralov muralov removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jul 19, 2023
@kyma-bot kyma-bot merged commit 80105ae into kyma-project:main Jul 19, 2023
@muralov muralov deleted the adapt-for-eventing-manager branch July 19, 2023 11:11
@kyma-bot
Copy link
Contributor

@contributor: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
mvshao_test_of_prowjob_pre-main-kyma-integration-k3d-app-gatewa 5b58f33 link false /test pre-main-kyma-integration-k3d-app-gateway

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@k15r k15r added the area/eventing Issues or PRs related to eventing label Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/eventing Issues or PRs related to eventing lgtm Looks good to me! size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Reconcile Eventing CR in NATS Mode
4 participants