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.

Sign up for GitHub

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

Jump to bottom

Remove BZ check which is migrated to Jira and Add new BZ check causing selinux denials #13923

Merged
merged 1 commit into from
Feb 13, 2024
Merged

Remove BZ check which is migrated to Jira and Add new BZ check causing selinux denials #13923

merged 1 commit into from
Feb 13, 2024

Conversation

shweta83
Copy link
Contributor

@shweta83 shweta83 commented Jan 29, 2024 •
edited
Loading

Problem Statement

The BZ in the test case has been migrated to jira so the check doesn't work.
The older BZ is entirely unrelated to the test case so removed it but we ran into another issue and raised a BZ for it -> BZ#2263294

Solution

Removing the check from the test.

Related Issues

@shweta83 shweta83 requested a review from a team as a code owner January 29, 2024 14:51
@shweta83 shweta83 added CherryPick PR needs CherryPick to previous branches 6.12.z Introduced in or relating directly to Satellite 6.12 AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing 6.13.z Introduced in or relating directly to Satellite 6.13 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 labels Jan 29, 2024
@shweta83 shweta83 changed the title Remove_NonExistent_BZ Remove BZ check which is migrated to Jira Jan 29, 2024
@shweta83
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/sys/test_pulp3_filesystem.py -k test_selinux_status

@shweta83
Copy link
Contributor Author

The test is failing with some unknown selinux denials and it is in discussion, so moving this PR to draft until the issue gets resolved.

@shweta83 shweta83 marked this pull request as draft January 31, 2024 13:20
@JacobCallahan
Copy link
Member

The test is failing with some unknown selinux denials and it is in discussion, so moving this PR to draft until the issue gets resolved.

Since this has been migrated to Jira, has the Jira issue been resolved? If not, then we'll need to block the test logic on that issue. This is something that @jameerpathan111 has volunteered to take on.

@shweta83
Copy link
Contributor Author

shweta83 commented Feb 1, 2024

The test is failing with some unknown selinux denials and it is in discussion, so moving this PR to draft until the issue gets resolved.

Since this has been migrated to Jira, has the Jira issue been resolved? If not, then we'll need to block the test logic on that issue. This is something that @jameerpathan111 has volunteered to take on.

The issue migrated to Jira is closed and it is a different issue from the test. I can block this issue but we need a new bug for it.

@shweta83 shweta83 force-pushed the fix_pulp_test branch 2 times, most recently from e37e049 to ebabb45 Compare February 9, 2024 09:32
@shweta83 shweta83 marked this pull request as ready for review February 9, 2024 09:33
@shweta83 shweta83 changed the title Remove BZ check which is migrated to Jira Remove BZ check which is migrated to Jira and Add new BZ check causing selinux denials Feb 13, 2024
@shweta83
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/sys/test_pulp3_filesystem.py -k test_selinux_status

@Gauravtalreja1 Gauravtalreja1 added the TestFailure Issues and PRs related to a test failing in automation label Feb 13, 2024
@Gauravtalreja1 Gauravtalreja1 enabled auto-merge (squash) February 13, 2024 12:28
@Gauravtalreja1 Gauravtalreja1 merged commit cf3de35 into SatelliteQE:master Feb 13, 2024
8 checks passed
github-actions bot pushed a commit that referenced this pull request Feb 13, 2024
@shweta83 @web-flow
Remove BZ check which is migrated to Jira and Add new BZ check causin…
e9e64ab 
…g selinux denials (#13923)

Remove_NonExistent_BZ

(cherry picked from commit cf3de35)
@Satellite-QE Satellite-QE mentioned this pull request Feb 13, 2024
Merged
github-actions bot pushed a commit that referenced this pull request Feb 13, 2024
@shweta83 @web-flow
Remove BZ check which is migrated to Jira and Add new BZ check causin…
14f6737 
…g selinux denials (#13923)

Remove_NonExistent_BZ

(cherry picked from commit cf3de35)
@Satellite-QE Satellite-QE mentioned this pull request Feb 13, 2024
Merged
github-actions bot pushed a commit that referenced this pull request Feb 13, 2024
@shweta83 @web-flow
Remove BZ check which is migrated to Jira and Add new BZ check causin…
c5e3278 
…g selinux denials (#13923)

Remove_NonExistent_BZ

(cherry picked from commit cf3de35)
@Satellite-QE Satellite-QE mentioned this pull request Feb 13, 2024
Merged
github-actions bot pushed a commit that referenced this pull request Feb 13, 2024
@shweta83 @web-flow
Remove BZ check which is migrated to Jira and Add new BZ check causin…
7c74434 
…g selinux denials (#13923)

Remove_NonExistent_BZ

(cherry picked from commit cf3de35)
@Satellite-QE Satellite-QE mentioned this pull request Feb 13, 2024
Merged
shweta83 added a commit to shweta83/robottelo that referenced this pull request Apr 10, 2024
@shweta83
Remove BZ check which is migrated to Jira and Add new BZ check causin…
fde5ed3 
…g selinux denials (SatelliteQE#13923)

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

@Gauravtalreja1 Gauravtalreja1 Gauravtalreja1 approved these changes

Assignees

@shweta83 shweta83

Labels
6.12.z Introduced in or relating directly to Satellite 6.12 6.13.z Introduced in or relating directly to Satellite 6.13 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing CherryPick PR needs CherryPick to previous branches TestFailure Issues and PRs related to a test failing in automation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@shweta83 @JacobCallahan @Gauravtalreja1