Skip to content

Build list of required jobs in generate-workflow#4326

Open
xrmx wants to merge 2 commits intoopen-telemetry:mainfrom
xrmx:build-jobs-for-opentelemetry-admin
Open

Build list of required jobs in generate-workflow#4326
xrmx wants to merge 2 commits intoopen-telemetry:mainfrom
xrmx:build-jobs-for-opentelemetry-admin

Conversation

@xrmx
Copy link
Contributor

@xrmx xrmx commented Mar 11, 2026

Description

Dump the list of all the jobs in the same format that is required for configuration in opentelemetry-admin terraform config files so that we don't miss required jobs.

Dump the list of all the jobs in the same format that is required for
configuration in opentelemetry-admin terraform config files so that
we don't miss required jobs.
@xrmx xrmx requested a review from a team as a code owner March 11, 2026 11:15
@xrmx xrmx added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Mar 11, 2026
Copy link
Member

Choose a reason for hiding this comment

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

Can you add a comment explaining what it's for?

Copy link
Member

@emdneto emdneto left a comment

Choose a reason for hiding this comment

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

Wondering if we can manage the required checks from our repo... Like, having a CI job that will open a PR in the admin repo with the updates when we want


commands =
python {toxinidir}/.github/workflows/generate_workflows.py
python {toxinidir}/.github/workflows/generate_required_checks.py
Copy link
Member

@emdneto emdneto Mar 11, 2026

Choose a reason for hiding this comment

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

Do you want to run that here?
If I understood correctly, you want to generate the input to populate the Terraform code on the admin repo right? Maybe this fits better on scripts dir, not sure if it's necessary to run in every tox -e generate-workflows

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip Changelog PRs that do not require a CHANGELOG.md entry

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

4 participants