Skip to content

Add step to build a "production" fedora build on pull requests - #15090

Open
Mab879 wants to merge 1 commit into
ComplianceAsCode:masterfrom
Mab879:add_prod_fedora_build
Open

Add step to build a "production" fedora build on pull requests#15090
Mab879 wants to merge 1 commit into
ComplianceAsCode:masterfrom
Mab879:add_prod_fedora_build

Conversation

@Mab879

@Mab879 Mab879 commented Sep 3, 2026

Copy link
Copy Markdown
Member

Description:

Add step to build a "production" fedora build on pull requests

Rationale:

To catch issues with packaging in upstream.

Review Hints:

Check that added job ran successfully.

@Mab879 Mab879 added this to the 0.1.83 milestone Sep 3, 2026
@packit-as-a-service

Copy link
Copy Markdown

Failed to load packit config file:

Cannot parse package config. ValidationError({'jobs': {0: {'job': ['Not a valid string.'], 'trigger': ['Missing data for required field.']}}})

For more info, please check out the documentation or contact the Packit team. You can also use our CLI command config validate or our pre-commit hooks for validation of the configuration.

@Mab879
Mab879 force-pushed the add_prod_fedora_build branch 4 times, most recently from bea3ac0 to f2d43ca Compare September 3, 2026 19:07
@Mab879
Mab879 force-pushed the add_prod_fedora_build branch 4 times, most recently from df65829 to fd7dcc8 Compare September 3, 2026 19:41
@Mab879
Mab879 force-pushed the add_prod_fedora_build branch from fd7dcc8 to 4ac1fb8 Compare September 3, 2026 20:30
Comment on lines +7 to +8
Version: 0.1.82
Release: 1%{?dist}

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.

I think it will build the version 0.1.82 forever.

In the other spec file, we have this:

# Version placeholder. Copr build version is determined by utils/version.sh. See .packit.yaml config
Version:	0.0.1
Release:	0%{?dist}

Try to have this thing also in this spec file.

@jan-cerny

Copy link
Copy Markdown
Member

/packit build

@jan-cerny jan-cerny self-assigned this Sep 4, 2026
@Mab879

Mab879 commented Sep 4, 2026

Copy link
Copy Markdown
Member Author

Just an FYI will have to switch up the names for the required builds due to this PR.

%define cmake_defines_common -DSSG_SEPARATE_SCAP_FILES_ENABLED=OFF -DSSG_BASH_SCRIPTS_ENABLED=OFF -DSSG_BUILD_SCAP_12_DS=OFF
%define cmake_defines_specific %{nil}
%if 0%{?rhel} && ! %{defined eln}
%define cmake_defines_specific -DSSG_PRODUCT_DEFAULT:BOOLEAN=FALSE -DSSG_PRODUCT_RHEL%{rhel}:BOOLEAN=TRUE -DSSG_SCIENTIFIC_LINUX_DERIVATIVES_ENABLED:BOOL=OFF -DSSG_CENTOS_DERIVATIVES_ENABLED:BOOL=OFF -DSSG_ANSIBLE_PLAYBOOKS_PER_RULE_ENABLED:BOOL=ON

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.

This seems to be yielding -DSSG_PRODUCT_RHEL46:BOOL=ON

%{rhel} is being set to 46 here according to: https://artifacts.dev.testing-farm.io/9c3be016-7b69-4e04-a739-6e5026e36de3/work-cixgdpip_f/plans/ci/execute/data/guest/default-0/static-checks/rpmbuild-ctest-1/output.txt

We probably need to hardcode the RHEL version here, or build all the products instead

Suggested change
%define cmake_defines_specific -DSSG_PRODUCT_DEFAULT:BOOLEAN=FALSE -DSSG_PRODUCT_RHEL%{rhel}:BOOLEAN=TRUE -DSSG_SCIENTIFIC_LINUX_DERIVATIVES_ENABLED:BOOL=OFF -DSSG_CENTOS_DERIVATIVES_ENABLED:BOOL=OFF -DSSG_ANSIBLE_PLAYBOOKS_PER_RULE_ENABLED:BOOL=ON
%define cmake_defines_specific -DSSG_PRODUCT_DEFAULT:BOOLEAN=FALSE -DSSG_SCIENTIFIC_LINUX_DERIVATIVES_ENABLED:BOOL=OFF -DSSG_CENTOS_DERIVATIVES_ENABLED:BOOL=OFF -DSSG_ANSIBLE_PLAYBOOKS_PER_RULE_ENABLED:BOOL=ON

option(SSG_PRODUCT_DEFAULT "If enabled, all default release products will be built; otherwise only explicitly enabled products will be" TRUE)

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.

No products seem to have been built

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