Skip to content

Add SLE16 to SUSE gating#14742

Open
Mab879 wants to merge 2 commits into
ComplianceAsCode:masterfrom
Mab879:add_sle16_to_workflow
Open

Add SLE16 to SUSE gating#14742
Mab879 wants to merge 2 commits into
ComplianceAsCode:masterfrom
Mab879:add_sle16_to_workflow

Conversation

@Mab879
Copy link
Copy Markdown
Member

@Mab879 Mab879 commented May 28, 2026

Description:

Add SLE16 to SUSE gating

Rationale:

Review Hints:

Double check that SLE16 was build in CI.

@Mab879 Mab879 added this to the 0.1.82 milestone May 28, 2026
@Mab879 Mab879 added the Infrastructure Our content build system label May 28, 2026
@Mab879
Copy link
Copy Markdown
Member Author

Mab879 commented May 28, 2026

This will fail until #14741 is merged.

uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v4
- name: Build
run: ./build_product sle12 sle15
run: ./build_product sle12 sle15 sle16
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It makes sense to change the name of the tests as well, probably to something more generic:

it contains reference to SLE 15 only Gate / Build, Test on SLE 15 (Container) (pull_request)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

actually the target testing container:

jobs:
  validate-sle:
    name: Build, Test on SLE 15 (Container)
    runs-on: ubuntu-latest
    container:
      image: registry.suse.com/bci/bci-base:latest
    steps:

in this case is image: registry.suse.com/bci/bci-base:latest which could be something higher than 15 I guess because of the latest, so changing to latest seems to be more accurate.

Mab879 and others added 2 commits May 29, 2026 12:05
It's using the following container image: registry.suse.com/bci/bci-base:latest
so it can be something later than SLE15.
@ggbecker ggbecker force-pushed the add_sle16_to_workflow branch from 60f9ca6 to af2d6f3 Compare May 29, 2026 10:12
@ggbecker
Copy link
Copy Markdown
Member

I have rebased on latest main branch after merging #14741 and added one more change to the title of the test.

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented May 29, 2026

@Mab879: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-aws-openshift-platform-compliance af2d6f3 link true /test e2e-aws-openshift-platform-compliance
ci/prow/e2e-aws-openshift-node-compliance af2d6f3 link true /test e2e-aws-openshift-node-compliance

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Infrastructure Our content build system

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants