Skip to content

Deployment for webhook dispatcher app#841

Merged
Allda merged 2 commits into
mainfrom
ISV-6109
Aug 13, 2025
Merged

Deployment for webhook dispatcher app#841
Allda merged 2 commits into
mainfrom
ISV-6109

Conversation

@Allda
Copy link
Copy Markdown
Contributor

@Allda Allda commented Aug 5, 2025

This commit brings a deployment configuration for a new webhook
dispatcher application.

It consist of following parts:

  • postgres database
  • webhook dispatcher application
  • services, routes, secrets, pvc, configs

The application deployment is fully driven by the Ansible.

As part of the commit I also split the "local" user workspace into
separate ansible inventory that share a data with stage but can be
customized.

JIRA: ISV-6109

Merge Request Checklists

  • Development is done in feature branches
  • Code changes are submitted as pull request into a primary branch [Provide reason for non-primary branch submissions]
  • Code changes are covered with unit and integration tests.
  • Code passes all automated code tests:
    • Linting
    • Code formatter - Black
    • Security scanners
    • Unit tests
    • Integration tests
  • Code is reviewed by at least 1 team member
  • Pull request is tagged with "risk/good-to-go" label for minor changes

Base automatically changed from ISV-6108 to main August 5, 2025 12:37
@Allda Allda force-pushed the ISV-6109 branch 3 times, most recently from a5e5c51 to 2414c3b Compare August 6, 2025 05:49
@Allda Allda requested review from ezopezo and kosciCZ August 6, 2025 09:45
Copy link
Copy Markdown
Contributor

@kosciCZ kosciCZ left a comment

Choose a reason for hiding this comment

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

LGTM overall. Based on my reading of the code, I think that it is missing some kind of communication back to the user in the PR about their place in the queue/if the trigger fails.

However this MR seems more about deployment, so this is probably out of scope for that.

@@ -107,7 +107,7 @@ security:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Typo in filename?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good catch, fixed.

Allda added 2 commits August 13, 2025 09:28
A dispatcher is a new component to manage incoming webhooks from Github.
It accept new requests and manage a queue based on cluster capacity.

A capacity is defined as a number of running pipelines in certain
namespace. This simple metrics will provide a buffer mechanism for time
periods when we receive a bunk releases of operators. The mechanism
keeps pending requests in the database queue and only trigger related
pipeline in case of free capacity.

The solution is made of:
- Rest API
- Postgres database
- Dispatcher
- Capacity manager

JIRA: ISV-6108

Signed-off-by: Ales Raszka <araszka@redhat.com>
This commit brings a deployment configuration for a new webhook
dispatcher application.

It consist of following parts:
 - postgres database
 - webhook dispatcher application
 - services, routes, secrets, pvc, configs

The application deployment is fully driven by the Ansible.

As part of the commit I also split the "local" user workspace into
separate ansible inventory that share a data with stage but can be
customized.

JIRA: ISV-6109

Signed-off-by: Ales Raszka <araszka@redhat.com>
@Allda Allda merged commit 127c0a4 into main Aug 13, 2025
4 checks passed
@Allda Allda deleted the ISV-6109 branch August 13, 2025 07:45
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.

3 participants