Skip to content

Infra Automation + Helm Chart Integration; CI Enhancements #992

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

Merged
merged 42 commits into from
Jul 14, 2025

Conversation

matoszz
Copy link
Member

@matoszz matoszz commented Jul 14, 2025

The original intent of this PR was to add functionality allowing for "sensitive" tagged fields to have programmatic kubernetes resources created for convenience, for consolidated configuration of domains via "domain prefix and suffix" conventions, automated Helm chart image bumps, versioning, and programmatic generation of the Helm values files via the same .config.yaml generation flows used today.

Since I was feeling 🐱 , I additionally added some stretch goals onto this PR for:

  • Build node disk space management and intelligence
  • Rich PR Details: Creates descriptive PRs with build information and change summary
  • Slack message integration for notification of created PR's easing the overall code to production flow
  • Docker + tool caching and pre-environment convenience helpers
  • Better ConfigMap formatting and generation
  • Automated ExternalSecret file generation and inclusion in PR process flow
  • "Draft" PR functionality to draft a view of the changes and then link them to the original PR (we're going to find out here in just a second if this works because this PR itself should ideally trigger a draft PR to be created against the openlane-infra repo, and that PR should be tagged in this one / linked together. Post-merge, that Draft PR should turn into a real one with any secrets, configurations, etc., added, otherwise the pipeline should run on image tag / release creation and automatically create a PR with the helm chart updated and send a slack message notifying the team of it)

…elm automation, rock the upgraded porche of config management
@matoszz matoszz requested a review from a team as a code owner July 14, 2025 04:47
@github-actions github-actions bot added enhancement New feature or request config ci labels Jul 14, 2025
@theopenlane-bender
Copy link

🔧 Configuration Changes Detected

This PR contains changes that will affect the Helm chart configuration. A draft infrastructure PR has been automatically created to preview these changes:

📋 Draft PR: theopenlane/openlane-infra#98

Changes Preview:


- 🔄 Merged Helm values.yaml
- 🆕 Created External Secrets templates
- ✨ Created ConfigMap template

The infrastructure PR will automatically convert from draft to ready for review once this core PR is merged.

@theopenlane-bender
Copy link

🔧 Configuration Changes Detected

This PR contains changes that will affect the Helm chart configuration. A draft infrastructure PR has been automatically created to preview these changes:

📋 Draft PR: theopenlane/openlane-infra#98

Changes Preview:


- 🔄 Merged Helm values.yaml
- ✅ Updated ConfigMap template

The infrastructure PR will automatically convert from draft to ready for review once this core PR is merged.

@theopenlane-bender
Copy link

🔧 Configuration Changes Detected

This PR contains changes that will affect the Helm chart configuration. A draft infrastructure PR has been automatically created to preview these changes:

📋 Draft PR: theopenlane/openlane-infra#98

Changes Preview:


- 🔄 Merged Helm values.yaml

The infrastructure PR will automatically convert from draft to ready for review once this core PR is merged.

@theopenlane-bender
Copy link

🔧 Configuration Changes Detected

This PR contains changes that will affect the Helm chart configuration. A draft infrastructure PR has been automatically created to preview these changes:

📋 Draft PR: theopenlane/openlane-infra#98

Changes Preview:


- 🔄 Merged Helm values.yaml
- ✅ Updated ConfigMap template

The infrastructure PR will automatically convert from draft to ready for review once this core PR is merged.

@theopenlane-bender
Copy link

🔧 Configuration Changes Detected

This PR contains changes that will affect the Helm chart configuration. A draft infrastructure PR has been automatically created to preview these changes:

📋 Draft PR: theopenlane/openlane-infra#98

Changes Preview:


- 🔄 Merged Helm values.yaml

The infrastructure PR will automatically convert from draft to ready for review once this core PR is merged.

Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 20%)

See analysis details on SonarQube Cloud

@golanglemonade golanglemonade merged commit ca96ff9 into main Jul 14, 2025
19 of 20 checks passed
@golanglemonade golanglemonade deleted the feat-tags branch July 14, 2025 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci config enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants