Skip to content
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

[ResponseOps][Task Manager] accept intervals with a PT prefix #204144

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

pmuellr
Copy link
Member

@pmuellr pmuellr commented Dec 12, 2024

Change task manager interval parsing to accept a PT prefix.

See: #204413

This will allow affected projects/deployments to get tasks running again, for those tasks whose intervals have been changed to have a PT prefix.

@pmuellr pmuellr added the ci:project-deploy-security Create a Security Serverless Project label Dec 12, 2024
@elasticmachine
Copy link
Contributor

elasticmachine commented Dec 12, 2024

🤖 Jobs for this PR can be triggered through checkboxes. 🚧

ℹ️ To trigger the CI, please tick the checkbox below 👇

  • Click to trigger kibana-pull-request for this PR!
  • Click to trigger kibana-deploy-project-from-pr for this PR!

@pmuellr pmuellr changed the title [DO NOT MERGE] trying to repro some Invalid interval "PT1M" errors [ResponseOps][Task Manager] accept intervals with a PT prefix Dec 18, 2024
@kibanamachine
Copy link
Contributor

PR Project deployment started at: https://buildkite.com/elastic/kibana-deploy-project-from-pr/builds/210

@kibanamachine
Copy link
Contributor

Project deployed, see credentials at: https://buildkite.com/elastic/kibana-deploy-project-from-pr/builds/210

@elasticmachine
Copy link
Contributor

elasticmachine commented Dec 18, 2024

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

✅ unchanged

History

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:project-deploy-security Create a Security Serverless Project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants