Skip to content

Update banner with rh01 mintmaker cronjob changes. - #13486

Open
gcpsoares wants to merge 2 commits into
redhat-appstudio:mainfrom
gcpsoares:rh01-mm-banner
Open

Update banner with rh01 mintmaker cronjob changes.#13486
gcpsoares wants to merge 2 commits into
redhat-appstudio:mainfrom
gcpsoares:rh01-mm-banner

Conversation

@gcpsoares

Copy link
Copy Markdown
Contributor

What

Update rh01 banner (PR #13392)

Why

Notify users.

Validation

kustomize build --enable-helm components/mintmaker/production/stone-prd-rh01/ passes and shows the banner updated.

Signed-off-by: Gabriel Soares <gasoares@redhat.com>
@github-actions

github-actions Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Kustomize Render Diff

Comparing 6ce29401f7858fb048

Component Environment Changes
components/konflux-info/production/stone-prd-rh01 production +3 -0

Total: 1 components, +3 -0 lines

📋 Full diff available in the workflow summary and as a downloadable artifact.

@qodo-for-redhat-appstudio

Copy link
Copy Markdown

PR Summary by Qodo

Update rh01 MintMaker schedule banner

⚙️ Configuration changes 🕐 Less than 10 minutes

Grey Divider

AI Description

• Add an informational banner announcing the reduced MintMaker dependency-check frequency on rh01.
• Ensure the new banner entry is first so it is the one displayed.
Diagram

graph TD
  A["banner-content.yaml"] --> B["Konflux Info deployment"] --> C["Cluster console UI"] --> D["Users"]
Loading
High-Level Assessment

The PR’s approach (updating the production banner content in-place) is the most direct and lowest-risk way to notify users about the schedule change.

Files changed (1) +2 / -1

Other (1) +2 / -1
banner-content.yamlAdd info banner for MintMaker schedule reduction +2/-1

Add info banner for MintMaker schedule reduction

• Prepends a new informational banner entry stating MintMaker dependency update checks on stone-prd-rh01 were reduced to twice daily (00:00 and 04:00 UTC). This ensures the message is the first (and therefore displayed) banner.

components/konflux-info/production/stone-prd-rh01/banner-content.yaml

@openshift-ci

openshift-ci Bot commented Aug 12, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: gcpsoares, sadlerap

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@qodo-for-redhat-appstudio

qodo-for-redhat-appstudio Bot commented Aug 12, 2026

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Tip of the day
💡 Did you know, you can tweak Display preferences with a live preview to see your comment before it ships

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Previous reviews

Review updated until commit bd2c098 ⚖️ Balanced

Results up to commit c5e7cc0 ⚖️ Balanced


No changes from previous review

Grey Divider

Qodo Logo

@codecov

codecov Bot commented Aug 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.38%. Comparing base (6ce2940) to head (bd2c098).
⚠️ Report is 18 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main   #13486   +/-   ##
=======================================
  Coverage   60.38%   60.38%           
=======================================
  Files          24       24           
  Lines        1628     1628           
=======================================
  Hits          983      983           
  Misses        563      563           
  Partials       82       82           
Flag Coverage Δ
go 60.38% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jkriz-rh

Copy link
Copy Markdown

Production Approval Record

Field Value
Action APPROVED
Reviewer @jkriz-rh
Timestamp 2026-08-13T09:20:08.097Z
Criteria Override Risk assessment not provided

Banner change only, there is no risk

@eedri

eedri commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

@gcpsoares this PR was approved for production 7d ago but is still open.

It has merge conflicts — please rebase or resolve them.

Once that's resolved it should merge; close the PR if it's no longer needed.

@openshift-ci openshift-ci Bot removed the lgtm label Aug 21, 2026
@openshift-ci

openshift-ci Bot commented Aug 21, 2026

Copy link
Copy Markdown

New changes are detected. LGTM label has been removed.

@qodo-for-redhat-appstudio

Copy link
Copy Markdown

Code review by qodo was updated up to the latest commit bd2c098

@gcpsoares

Copy link
Copy Markdown
Contributor Author

Conflict resolved, not sure yet how banner will behave with the 2 msgs.

@jkriz-rh

Copy link
Copy Markdown

Production Approval Record

Field Value
Action APPROVED
Reviewer @jkriz-rh
Timestamp 2026-08-24T08:51:30.819Z
Criteria Override Risk assessment not provided

Banner change only, has low risk

@eedri

eedri commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

@gcpsoares this PR was approved for production 4d ago but is still open.

Some checks are still failing — please get them green.

Once that's resolved it should merge; close the PR if it's no longer needed.

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.

4 participants