Skip to content
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

🤖 Triggering CI on branch 'release-next' after synching to upstream/main #423

Merged
merged 1 commit into from
Nov 6, 2021

Conversation

alanfx
Copy link
Collaborator

@alanfx alanfx commented Nov 6, 2021

No description provided.

@openshift-ci
Copy link

openshift-ci bot commented Nov 6, 2021

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: alanfx

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@alanfx
Copy link
Collaborator Author

alanfx commented Nov 6, 2021

OCF Webhook is merging this PR

@alanfx alanfx merged commit 7ec10b1 into release-next Nov 6, 2021
ReToCode pushed a commit to ReToCode/eventing-kafka that referenced this pull request Jan 3, 2023
…ive#423)

* Rename some import aliases before creating another common pkg

* Move some structs to common package

* Fix build

* Add missing license header

* Move kafka-config map watcher to common pkg

* Move code from channel/distr/common/testing to /common/testing

* Move more from channel/distr/common/testing to /common/testing

* Rename InitializeConfigWatcher->InitializeKafkaConfigMapWatcher

* Use common method to watch config-kafka in consolidated contrlr

* Use the already existing CM watcher, or create it otherwise

* Use the already existing CM watcher, or create it otherwise

* Don't start the cm watcher again
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant