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

add kafka eventbus spec #12

Merged
merged 5 commits into from
Jan 18, 2023
Merged

add kafka eventbus spec #12

merged 5 commits into from
Jan 18, 2023

Conversation

premadk
Copy link

@premadk premadk commented Jan 17, 2023

Signed-off-by: Prema devi Kuppuswamy premadk@gmail.com

Checklist:

premadk and others added 2 commits January 17, 2023 09:46
Signed-off-by: Prema devi Kuppuswamy <premadk@gmail.com>
pkg/apis/eventbus/v1alpha1/kafka_eventbus.go Outdated Show resolved Hide resolved
@@ -57,3 +57,10 @@ data:
metricsExporterImage: natsio/prometheus-nats-exporter:0.9.1
configReloaderImage: natsio/nats-server-config-reloader:0.7.0
startCommand: /nats-server
kafka:
streamConfig: |
# The default properties of the streams to be created in this Kafka service
Copy link

@dfarr dfarr Jan 18, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# The default properties of the streams to be created in this Kafka service
# Default Kafka settings, can be overridden by EventBus Kafka specs

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just changing this to align with the jetstream comment

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that's for the settings of jetstream.. jetstream's stream config have this one "# The default properties of the streams to be created in this JetStream service" thats already in sync..
but i still feel it need to be rephrased

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Whoops, updated my comment

@@ -351,6 +351,13 @@ data:
metricsExporterImage: natsio/prometheus-nats-exporter:0.9.1
configReloaderImage: natsio/nats-server-config-reloader:0.7.0
startCommand: /nats-server
kafka:
streamConfig: |
# The default properties of the streams to be created in this Kafka service
Copy link

@dfarr dfarr Jan 18, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# The default properties of the streams to be created in this Kafka service
# Default Kafka settings, can be overridden by EventBus Kafka specs

@@ -271,6 +271,13 @@ data:
metricsExporterImage: natsio/prometheus-nats-exporter:0.9.1
configReloaderImage: natsio/nats-server-config-reloader:0.7.0
startCommand: /nats-server
kafka:
streamConfig: |
# The default properties of the streams to be created in this Kafka service
Copy link

@dfarr dfarr Jan 18, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# The default properties of the streams to be created in this Kafka service
# Default Kafka settings, can be overridden by EventBus Kafka specs

pkg/apis/eventbus/v1alpha1/kafka_eventbus.go Outdated Show resolved Hide resolved
premadk and others added 3 commits January 18, 2023 11:14
Signed-off-by: Prema devi Kuppuswamy <premadk@gmail.com>
Signed-off-by: Prema devi Kuppuswamy <premadk@gmail.com>
Copy link

@dfarr dfarr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@dfarr dfarr merged commit e8d4117 into feature/kafka-eventbus Jan 18, 2023
@dfarr dfarr deleted the busspec branch January 18, 2023 20:06
dfarr pushed a commit that referenced this pull request Feb 1, 2023
Signed-off-by: Prema devi Kuppuswamy <premadk@gmail.com>
Signed-off-by: David Farr <david_farr@intuit.com>
dfarr pushed a commit that referenced this pull request Feb 16, 2023
Signed-off-by: Prema devi Kuppuswamy <premadk@gmail.com>
Signed-off-by: David Farr <david_farr@intuit.com>
dfarr pushed a commit that referenced this pull request Mar 2, 2023
Signed-off-by: Prema devi Kuppuswamy <premadk@gmail.com>
Signed-off-by: David Farr <david_farr@intuit.com>
dfarr pushed a commit that referenced this pull request Mar 3, 2023
Signed-off-by: Prema devi Kuppuswamy <premadk@gmail.com>
Signed-off-by: David Farr <david_farr@intuit.com>
dfarr pushed a commit that referenced this pull request Mar 3, 2023
Signed-off-by: Prema devi Kuppuswamy <premadk@gmail.com>
Signed-off-by: David Farr <david_farr@intuit.com>
dfarr pushed a commit that referenced this pull request Mar 3, 2023
Signed-off-by: Prema devi Kuppuswamy <premadk@gmail.com>
Signed-off-by: David Farr <david_farr@intuit.com>
dfarr pushed a commit that referenced this pull request Mar 16, 2023
Signed-off-by: Prema devi Kuppuswamy <premadk@gmail.com>
Signed-off-by: David Farr <david_farr@intuit.com>
dfarr pushed a commit that referenced this pull request Mar 21, 2023
Signed-off-by: Prema devi Kuppuswamy <premadk@gmail.com>
Signed-off-by: David Farr <david_farr@intuit.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants