Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(RHEL-1087) test: backport TEST-81-GENERATORS (fstab-generator only) #417

Merged
merged 3 commits into from
Nov 28, 2023

Conversation

mrc0mmand
Copy link
Member

@mrc0mmand mrc0mmand commented Nov 27, 2023

Some fstab-generator features are not present on RHEL 8 or they behave
differently - in such case there's an inline comment explaining what's
different with a reference to an upstream commit that introduced the
changed behavior.

Realated: https://issues.redhat.com/browse/RHEL-1087

rhel-only


Given the recent issue regarding fstab-generator, let's backport the fstab-genreator part of TEST-81-GENERATORS from upstream to make sure we'd catch such issues in the future.

Copy link

github-actions bot commented Nov 27, 2023

Tracker - RHEL-1087

The following commits meet all requirements

commit upstream
4cb7355 - fstab-generator: allow overriding /etc/fstab with $SYSTEMD_FSTAB rhel-only
7400723 - fstab-generator: allow overriding path to /sysroot/etc/fstab too rhel-only
9e50ca6 - test: backport TEST-81-GENERATORS (fstab-generator only) rhel-only

@github-actions github-actions bot added pr/needs-ci Formerly needs-ci pr/needs-review Formerly needs-review labels Nov 27, 2023
Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

shellcheck found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.

@github-actions github-actions bot added the tracker/missing Formerly needs-bz label Nov 27, 2023
Based on: 99e3d4767932bce5febb45e8543162d729d17425
Related: RHEL-1087

rhel-only
@mrc0mmand mrc0mmand force-pushed the generator-tests branch 2 times, most recently from 211bbd6 to b0309a0 Compare November 27, 2023 15:51
@github-actions github-actions bot removed the tracker/missing Formerly needs-bz label Nov 27, 2023
@github-actions github-actions bot changed the title test: backport TEST-81-GENERATORS (fstab-generator only) (RHEL-1087) test: backport TEST-81-GENERATORS (fstab-generator only) Nov 27, 2023
Some fstab-generator features are not present on RHEL 8 or they behave
differently - in such case there's an inline comment explaining what's
different with a reference to an upstream commit that introduced the
changed behavior.

Related: RHEL-1087

rhel-only
@github-actions github-actions bot removed the pr/needs-ci Formerly needs-ci label Nov 27, 2023
Copy link
Member

@dtardon dtardon left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions github-actions bot removed the pr/needs-review Formerly needs-review label Nov 28, 2023
@jamacku jamacku added this to the RHEL-8.10 milestone Nov 28, 2023
@jamacku jamacku merged commit 5202603 into redhat-plumbers:main Nov 28, 2023
12 of 13 checks passed
@mrc0mmand mrc0mmand deleted the generator-tests branch November 28, 2023 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants