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

simplify time expression assertions #14379

Merged
merged 1 commit into from
Mar 13, 2024
Merged

simplify time expression assertions #14379

merged 1 commit into from
Mar 13, 2024

Conversation

pondrejk
Copy link
Contributor

Removing one expression causing flakiness in this test

@pondrejk pondrejk added CherryPick PR needs CherryPick to previous branches 6.15.z Introduced in or relating directly to Satellite 6.15 labels Mar 13, 2024
@pondrejk pondrejk self-assigned this Mar 13, 2024
@pondrejk pondrejk requested a review from a team as a code owner March 13, 2024 09:58
@pondrejk
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_remoteexecution.py -k time

@pondrejk pondrejk changed the title simplify time experssion assertions simplify time expression assertions Mar 13, 2024
@pondrejk pondrejk force-pushed the time-experssions-amend branch from 1cd5337 to 6ba3d78 Compare March 13, 2024 11:56
@pondrejk
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_remoteexecution.py -k time

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 6044
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/cli/test_remoteexecution.py -k time --external-logging
Test Result : =========== 1 passed, 65 deselected, 1 warning in 1007.29s (0:16:47) ===========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Mar 13, 2024
@Gauravtalreja1 Gauravtalreja1 added Easy Fix :) Easiest Fix to review and quick merge request. AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing labels Mar 13, 2024
@Gauravtalreja1 Gauravtalreja1 enabled auto-merge (squash) March 13, 2024 13:33
@Gauravtalreja1 Gauravtalreja1 merged commit 57a505a into SatelliteQE:master Mar 13, 2024
11 checks passed
github-actions bot pushed a commit that referenced this pull request Mar 13, 2024
@pondrejk @web-flow
simplify time expression assertions (#14379)
045f3c4 
simplify time experssion assertions

(cherry picked from commit 57a505a)
@Satellite-QE Satellite-QE mentioned this pull request Mar 13, 2024
Merged
shweta83 pushed a commit to shweta83/robottelo that referenced this pull request Apr 10, 2024
@pondrejk @shweta83
simplify time expression assertions (SatelliteQE#14379)
5e1e240 
simplify time experssion assertions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shubhamsg199 shubhamsg199 shubhamsg199 approved these changes

@Gauravtalreja1 Gauravtalreja1 Gauravtalreja1 approved these changes

Assignees

@pondrejk pondrejk

Labels
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 Easy Fix :) Easiest Fix to review and quick merge request. PRT-Passed Indicates that latest PRT run is passed for the PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@pondrejk @Satellite-QE @Gauravtalreja1 @shubhamsg199