Skip to content

Yaml files cron timer eol - #507

Draft
jose-caballero wants to merge 3 commits into
mainfrom
yaml_files_cron_timer_eol
Draft

Yaml files cron timer eol#507
jose-caballero wants to merge 3 commits into
mainfrom
yaml_files_cron_timer_eol

Conversation

@jose-caballero

Copy link
Copy Markdown
Contributor

Skeleton for the new hygiene workflow. Here we create the yaml files and a fake implementation of the actual automation code.

@codecov

codecov Bot commented Aug 26, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 98.99%. Comparing base (59bf3cc) to head (09b7ef3).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
lib/workflows/manage_eol_servers.py 0.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #507      +/-   ##
==========================================
- Coverage   99.10%   98.99%   -0.11%     
==========================================
  Files         112      113       +1     
  Lines        2787     2790       +3     
  Branches      340      340              
==========================================
  Hits         2762     2762              
- Misses         22       25       +3     
  Partials        3        3              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

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

We are going to trigger the hygiene automation for EOL VMs from a cron
(st2 cronTimer).
Here we create the cron YAML config file.

In this config we set the OS names and versions we want to automation to
pay attention to, instead of hardcoding it in the code.
Here we create the cron YAML config file for the ACTION.
It must include the parameters from the cron YAML file.
@jose-caballero
jose-caballero force-pushed the yaml_files_cron_timer_eol branch from 23cb9a0 to 09b7ef3 Compare August 27, 2026 08:27
type: core.st2.CronTimer
parameters:
timezone: "Europe/London"
day_of_week: mon,tue,wed,thu,fri

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I don't think this need to run daily, as not much is going to change day to day regarding OS version, maybe once a week?

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.

Hmm... The point was to detect them as soon as possible, but w/o going too fast. I think I will add this question to the others for David F.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think daily is fine. ST2 is supposed to be an automation tool so let it automate away.

@jose-caballero
jose-caballero marked this pull request as draft August 27, 2026 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants