Skip to content

Add ability to disable maintenance via component#668

Open
Kidswiss wants to merge 3 commits intodevelopfrom
add/disable_maintenance
Open

Add ability to disable maintenance via component#668
Kidswiss wants to merge 3 commits intodevelopfrom
add/disable_maintenance

Conversation

@Kidswiss
Copy link
Copy Markdown
Contributor

@Kidswiss Kidswiss commented May 6, 2026

Summary

  • This is handled completely within the job itself, circumventing the release management
  • A new configmap is deployed to the appcat-control namespace which will contain configuration for each service
  • If the configmap or any service is not defined it will assume everything is enabled, otherwise take the settings from the configmap

Checklist

  • Update tests.
  • Link this PR to related issues.
  • Merge with /merge comment.

Component PR: vshn/component-appcat#1174

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 7, 2026

@Kidswiss Kidswiss force-pushed the add/disable_maintenance branch from 1a4bece to 76800b8 Compare May 7, 2026 07:43
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 7, 2026

@Kidswiss Kidswiss force-pushed the add/disable_maintenance branch from 76800b8 to cdb6220 Compare May 7, 2026 07:56
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 7, 2026

@Kidswiss Kidswiss force-pushed the add/disable_maintenance branch from cdb6220 to c273345 Compare May 7, 2026 13:07
Kidswiss added 2 commits May 7, 2026 13:07
pkg/maintenance/config.go + tests. Reads <service>.disableServiceRelease
/ <service>.disableAppcatRelease from named ConfigMap.
cmd/maintenance.go reads ConfigMap at runtime, ORs with existing
DISABLE_APPCAT_RELEASE env var, gates both appcat release and service
release phases.
@Kidswiss Kidswiss force-pushed the add/disable_maintenance branch from c273345 to 3d1c651 Compare May 7, 2026 13:07
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 7, 2026

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 7, 2026

…maintenance cronjob

Composition function adds env vars so maintenance container knows where
to find ConfigMap.
@Kidswiss Kidswiss force-pushed the add/disable_maintenance branch from 3d1c651 to 5fb3e6a Compare May 7, 2026 13:29
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 7, 2026

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant