Skip to content

[CRE][DONTime] Restore rollout flag#1991

Merged
bolekk merged 1 commit intomainfrom
dontime_flag_fix
Apr 20, 2026
Merged

[CRE][DONTime] Restore rollout flag#1991
bolekk merged 1 commit intomainfrom
dontime_flag_fix

Conversation

@bolekk
Copy link
Copy Markdown
Contributor

@bolekk bolekk commented Apr 20, 2026

Even though the flag is not used in Outcome any more, it still needs to be set in the Observation phase so nodes running old versions won't revert back during upgrade.

@bolekk bolekk requested a review from a team as a code owner April 20, 2026 20:51
@github-actions
Copy link
Copy Markdown

👋 bolekk, thanks for creating this pull request!

To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team.

Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks!

@github-actions
Copy link
Copy Markdown

✅ API Diff Results - github.com/smartcontractkit/chainlink-common

✅ Compatible Changes (2)

pkg/workflows/dontime/pb.(*Observation) (1)
  • GetPruneExecutions — ➕ Added
pkg/workflows/dontime/pb.Observation (1)
  • PruneExecutions — ➕ Added

📄 View full apidiff report

@bolekk bolekk added this pull request to the merge queue Apr 20, 2026
Merged via the queue into main with commit f5928e7 Apr 20, 2026
31 of 33 checks passed
@bolekk bolekk deleted the dontime_flag_fix branch April 20, 2026 21:32
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.

4 participants