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

feat: Adding Pulsar sensor #1356

Merged
merged 2 commits into from
Sep 23, 2021
Merged

feat: Adding Pulsar sensor #1356

merged 2 commits into from
Sep 23, 2021

Conversation

arnarg
Copy link
Contributor

@arnarg arnarg commented Sep 23, 2021

Checklist:

Adds a Pulsar sensor trigger so you can send messages to a Pulsar topic.

@arnarg arnarg changed the title Adding Pulsar sensor feat: Adding Pulsar sensor Sep 23, 2021
@arnarg arnarg marked this pull request as ready for review September 23, 2021 13:56
Copy link
Member

@whynowy whynowy left a comment

Choose a reason for hiding this comment

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

It's great to see a Pulsar trigger implementation, thanks for your contribution!

Would you like to come up with dome docs under https://github.com/argoproj/argo-events/tree/master/docs/sensors/triggers, it could be either in this PR, or a follow up one.

sensors/triggers/pulsar/pulsar.go Outdated Show resolved Hide resolved
Signed-off-by: Arnar Gauti Ingason <[email protected]>
@arnarg
Copy link
Contributor Author

arnarg commented Sep 23, 2021

@whynowy I will create some documentation tomorrow :)

@arnarg
Copy link
Contributor Author

arnarg commented Sep 23, 2021

I have added some docs for the Pulsar trigger. It's a copy of the Kafka trigger docs adapted to my new Pulsar one.

@arnarg arnarg requested a review from whynowy September 23, 2021 19:23
Copy link
Member

@whynowy whynowy left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

Copy link
Member

@whynowy whynowy left a comment

Choose a reason for hiding this comment

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

LGTM

@whynowy whynowy merged commit 74205b0 into argoproj:master Sep 23, 2021
@arnarg arnarg deleted the pulsar-sensor-trigger branch September 24, 2021 05:44
aslafy-z pushed a commit to aslafy-z/argo-events that referenced this pull request Oct 6, 2021
* feat: Adding Pulsar sensor

Signed-off-by: Arnar Gauti Ingason <[email protected]>

* feat: Adding Pulsar trigger documentation

Signed-off-by: Arnar Gauti Ingason <[email protected]>
Signed-off-by: GitHub <[email protected]>
juliev0 pushed a commit to juliev0/argo-events that referenced this pull request Mar 29, 2022
* feat: Adding Pulsar sensor

Signed-off-by: Arnar Gauti Ingason <[email protected]>

* feat: Adding Pulsar trigger documentation

Signed-off-by: Arnar Gauti Ingason <[email protected]>
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