Skip to content

fix(ci): rename continuation config parameters with c- prefix#19142

Merged
theochap merged 1 commit intodevelopfrom
push-xrutkokwvusv
Feb 11, 2026
Merged

fix(ci): rename continuation config parameters with c- prefix#19142
theochap merged 1 commit intodevelopfrom
push-xrutkokwvusv

Conversation

@theochap
Copy link
Member

@theochap theochap commented Feb 10, 2026

Summary

  • Rename all pipeline parameters in CircleCI continuation configs (main.yml, rust-ci.yml, rust-e2e.yml) to use a c- prefix, eliminating naming overlap with the setup config (config.yml)
  • This fixes the {"message":"Conflicting pipeline parameters."} error that occurs when API-triggered pipelines (e.g. from opgitgovernance bot) pass non-default parameter values
  • The path-filtering mapping in config.yml bridges between the setup parameter names (used by API triggers) and the new c- prefixed continuation parameter names

Context

CircleCI's dynamic configuration has a limitation: when the same pipeline parameter name is defined in both the setup config and the continuation config, API-triggered pipelines fail if they pass a non-default value for that parameter. Webhook and scheduled triggers are unaffected because their parameters stay at default values.

This is a known CircleCI issue discussed in their forums:

Test plan

  • Verify that webhook-triggered pipelines still work (parameters passed through mapping at default values)
  • Verify that API-triggered pipelines from opgitgovernance bot no longer fail with "Conflicting pipeline parameters"
  • Verify that manual dispatch from CircleCI UI works for all *_dispatch parameters

🤖 Generated with Claude Code

@theochap theochap requested a review from a team as a code owner February 10, 2026 22:52
@theochap theochap requested a review from teddyknox February 10, 2026 22:52
@almanax-ai
Copy link
Contributor

almanax-ai bot commented Feb 10, 2026

Plan expired

Your subscription has expired. Please renew your subscription to continue using CI/CD integration and other features.

@codecov
Copy link

codecov bot commented Feb 10, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.0%. Comparing base (e6ce426) to head (8f9569d).
⚠️ Report is 3 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #19142      +/-   ##
===========================================
+ Coverage     75.7%    76.0%    +0.2%     
===========================================
  Files          692      188     -504     
  Lines        71528    10946   -60582     
===========================================
- Hits         54217     8324   -45893     
+ Misses       17165     2478   -14687     
+ Partials       146      144       -2     
Flag Coverage Δ
cannon-go-tests-64 66.4% <ø> (-0.9%) ⬇️
contracts-bedrock-tests 81.6% <ø> (-0.1%) ⬇️
unit ?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 510 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Member

@sebastianst sebastianst left a comment

Choose a reason for hiding this comment

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

CircleCI parameters are really a mess to deal with. I'm wondering if there's a better way to pass through params.

@theochap theochap enabled auto-merge February 11, 2026 15:06
@theochap theochap added this pull request to the merge queue Feb 11, 2026
@theochap theochap removed this pull request from the merge queue due to a manual request Feb 11, 2026
Rename all pipeline parameters in CircleCI continuation configs
(main.yml, rust-ci.yml, rust-e2e.yml) to use a `c-` prefix, eliminating
naming overlap with the setup config (config.yml). This fixes the
"Conflicting pipeline parameters" error that occurs when API-triggered
pipelines (e.g. from opgitgovernance bot) pass non-default parameter
values.

The path-filtering mapping in config.yml bridges between the setup
parameter names (used by API triggers) and the new continuation
parameter names.
@theochap theochap enabled auto-merge February 11, 2026 16:11
@theochap theochap added this pull request to the merge queue Feb 11, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 11, 2026
@theochap theochap added this pull request to the merge queue Feb 11, 2026
Merged via the queue into develop with commit b5c24a6 Feb 11, 2026
81 checks passed
@theochap theochap deleted the push-xrutkokwvusv branch February 11, 2026 18:12
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.

2 participants