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.

Sign up for GitHub

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

Jump to bottom

Add event controllers #8

Merged
merged 3 commits into from
May 24, 2024
Merged

Add event controllers #8

merged 3 commits into from
May 24, 2024

Conversation

daurer
Copy link
Contributor

@daurer daurer commented May 17, 2024 •
edited
Loading

  • Argo event controllers
  • EventBus deployments
  • EventSource deployments

The EventBus deployments are not deployed with the argo-events helm chart: argoproj/argo-helm#840

@daurer daurer marked this pull request as ready for review May 20, 2024 14:01
@daurer daurer requested a review from garryod May 20, 2024 14:25
@garryod garryod added enhancement New feature or request helm Pull request that updates Helm charts labels May 20, 2024
@daurer daurer force-pushed the events branch 2 times, most recently from 8f8fa12 to a144adc Compare May 20, 2024 14:36
garryod
garryod requested changes May 20, 2024
View reviewed changes
README.md Outdated Show resolved Hide resolved
charts/events/charts/argo-events-2.4.4.tgz Outdated Show resolved Hide resolved
charts/events/templates/event-bus.yaml Outdated Show resolved Hide resolved
charts/events/templates/sensor-service-account.yaml Outdated Show resolved Hide resolved
charts/events/templates/workflow-role-binding.yaml Outdated Show resolved Hide resolved
charts/events/values.yaml Outdated Show resolved Hide resolved
docs/index.md Outdated Show resolved Hide resolved
@daurer daurer force-pushed the events branch 3 times, most recently from 81fe190 to c762217 Compare May 20, 2024 16:09
garryod
garryod requested changes May 21, 2024
View reviewed changes
charts/events/templates/event-sources.yaml Outdated Show resolved Hide resolved
charts/events/templates/event-sources.yaml Outdated Show resolved Hide resolved
docs/how-tos/trigger-with-webhooks.md Outdated Show resolved Hide resolved
@daurer daurer force-pushed the events branch 2 times, most recently from 55be123 to 88b778d Compare May 21, 2024 14:13
garryod
garryod requested changes May 22, 2024
View reviewed changes
charts/events/templates/event-sources.yaml Outdated Show resolved Hide resolved
docs/how-tos/trigger-with-webhooks.md Show resolved Hide resolved
garryod
garryod requested changes May 23, 2024
View reviewed changes
charts/events/templates/event-sources.yaml Outdated Show resolved Hide resolved
charts/events/values.yaml Show resolved Hide resolved
mkdocs.yml Outdated Show resolved Hide resolved
@daurer daurer force-pushed the events branch 3 times, most recently from fd9e983 to e67e34c Compare May 24, 2024 08:14
@daurer
Add helm charts and templates for deploying events
bda1aaa 
Added eventbus and event source deployments

Updated README

Add docs for triggering workflows with webhook sensor

Improve templating for event-sources
@daurer
Publish events chart as part of CI
859e33b
@daurer
Added a note to the how-to guide
eb34796
garryod
garryod approved these changes May 24, 2024
View reviewed changes
Copy link
Member

@garryod garryod left a comment

Choose a reason for hiding this comment

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

LGTM

@daurer daurer merged commit 1508310 into main May 24, 2024
12 checks passed
@daurer daurer deleted the events branch May 24, 2024 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@garryod garryod garryod approved these changes

Assignees

@daurer daurer

Labels
enhancement New feature or request helm Pull request that updates Helm charts
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@daurer @garryod