Skip to content

Conversation

@ben-c-at-moz
Copy link
Collaborator

@ben-c-at-moz ben-c-at-moz commented Nov 14, 2025

Relevant Links

Bugzilla

Description of Code / Doc Changes

  • Modify choose_ci_set to read manifests if set in STARFOX_MANIFEST env var
  • Create manifests for CI and incident
  • Modify GHA pipeline to run incident manifest on manual execution
  • Rider: change the way we gather firefox version
  • Lint

Process Changes Required

Mark the relevant boxes:

  • Changes CI flow

Screenshots or Explanations

YAML schemas that are valid:

  • suite

    • test: status
  • suite

    • test
      • os1: status
      • os2: other_status
  • suite

    • test
      • subtest: status
  • suite

    • test
      • subtest
        • os1: status
        • os2: other_status

Comments or Future Work

  • A script to generate new manifests
  • Implementing a smoke manifest / functional manifests (rotations)
  • Wiring the smoke and functional manifests into the scheduled runs
  • Ensuring changes to how we gather version in conftest don't break scheduled runs

Workflow Checklist

  • Please request reviewers
  • If this is an unblocker, please post in Slack.
  • If asked to address comments, please resolve conversations.
  • If asked to change code, please re-request review from the person who wanted changes.

Thank you!

Copy link
Collaborator

@Tracy-Walker Tracy-Walker left a comment

Choose a reason for hiding this comment

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

comment completion and conflict heads up. otherwise looks good.

Copy link
Collaborator

@Tracy-Walker Tracy-Walker left a comment

Choose a reason for hiding this comment

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

let's go

@ben-c-at-moz ben-c-at-moz merged commit b2c6d44 into main Nov 24, 2025
44 of 45 checks passed
@ben-c-at-moz ben-c-at-moz deleted the ben/new-manifests branch November 24, 2025 18:03
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