Skip to content

Name the appliance check for what it proves - #12

Merged
marcos-mendez merged 1 commit into
mainfrom
ci/name-the-appliance-check
Sep 29, 2026
Merged

marcos-mendez merged 1 commit into
mainfrom
ci/name-the-appliance-check

Conversation

@marcos-mendez

@marcos-mendez marcos-mendez commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

Closes #11. Follows Keel-Linux/.github#12, which renames the reusable
job build-and-boot to boot-published-layer and makes a layer that has never
been published fail the job instead of passing it.

Comments and documentation only: nothing about this repository's behaviour
changes here. The rename takes effect the moment that pull request merges,
because this caller uses @main.

Branch protection on this repository's default branch requires the old check
name and must be changed to the new one when that merges. GitHub fails
closed
, so the cost of forgetting is a locked repository and not an
unguarded one: a required context that is never reported leaves every pull
request blocked, waiting for a status that will never arrive. Measured on
keel-mariadb on 2026-09-28, two pull requests under the same rule with every
reported check green: number 16, reporting appliance / build-and-boot, was
clean; number 17, reporting appliance / boot-published-layer and identical
otherwise, was blocked. Renaming the rule is a maintainer step and is listed
in the body of Keel-Linux/.github#12 with the other repositories in the same
position.

keel-linux/.github renames the reusable job build-and-boot to
boot-published-layer, because the job boots the layer the mirror
publishes and not the branch under test, and makes a layer that has
never been published fail instead of pass. The comments and the
documentation here named the old check and described the old behaviour.
@marcos-mendez
marcos-mendez merged commit a359d68 into main Sep 29, 2026
3 checks passed
@marcos-mendez
marcos-mendez deleted the ci/name-the-appliance-check branch September 29, 2026 03:20
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.

The appliance check is named for a boot of this branch that never happens

1 participant