Skip to content

Conversation

@myakove
Copy link
Collaborator

@myakove myakove commented Sep 3, 2023

wait_for_condition() cal self.instance and the resource me not be exists, add wait for the resource

@redhat-qe-bot
Copy link
Contributor

Report bugs in Issues

The following are automatically added:

  • Add reviewers from OWNER file (in the root of the repository) under reviewers section.
  • Set PR size label.
  • New issue is created for the PR. (Closed when PR is merged/closed)
  • Run pre-commit if .pre-commit-config.yaml exists in the repo.

Available user actions:

  • To mark PR as WIP comment /wip to the PR, To remove it from the PR comment /wip cancel to the PR.
  • To block merging of PR comment /hold, To un-block merging of PR comment /hold cancel.
  • To mark PR as verified comment /verified to the PR, to un-verify comment /verified cancel to the PR.
    verified label removed on each new commit push.
  • To cherry pick a merged PR comment /cherry-pick <target branch to cherry-pick to> in the PR.
    • Multiple target branches can be cherry-picked, separated by spaces. (/cherry-pick branch1 branch2)
    • Cherry-pick will be started when PR is merged
  • To re-run tox comment /retest tox in the PR.
  • To re-run build-container command /retest build-container in the PR.
  • To re-run python-module-install command /retest python-module-install in the PR.
  • To build and push container image command /build-and-push-container in the PR (tag will be the PR number).
  • To add a label by comment use /<label name>, to remove, use /<label name> cancel
Supported labels
  • hold
  • verified
  • wip
  • lgtm

@myakove
Copy link
Collaborator Author

myakove commented Sep 3, 2023

/cherry-pick v4.14 v4.13

@redhat-qe-bot
Copy link
Contributor

Cherry-pick requested for PR: Wait for resource exists before sampling for condition by user myakove
Adding label/s cherry-pick-v4.14 cherry-pick-v4.13 for automatic cheery-pick once the PR is merged

@myakove
Copy link
Collaborator Author

myakove commented Sep 3, 2023

/verified

@myakove myakove merged commit 2792ab5 into main Sep 3, 2023
@myakove myakove deleted the wait_for_condition-wait-for-exists branch September 3, 2023 10:49
redhat-qe-bot pushed a commit that referenced this pull request Sep 3, 2023
* Wait for resource exists before sampling for condition

* return None in self.instance

* TimeoutSampler: rempve dulicate exception log
@redhat-qe-bot
Copy link
Contributor

Cherry-picked PR Wait for resource exists before sampling for condition into v4.14

@redhat-qe-bot
Copy link
Contributor

Manual cherry-pick is needed
Cherry pick failed for 2792ab5 to v4.13:
To cherry-pick run:

git remote update
git checkout v4.13
git pull origin v4.13
git checkout -b wait_for_condition-wait-for-exists-v4.13
git cherry-pick 2792ab57386a0cacdc3e9526eb41210f63087cfd
git push origin wait_for_condition-wait-for-exists-v4.13

redhat-qe-bot added a commit that referenced this pull request Sep 3, 2023
* Wait for resource exists before sampling for condition

* return None in self.instance

* TimeoutSampler: rempve dulicate exception log

Co-authored-by: Meni Yakove <441263+myakove@users.noreply.github.com>
myakove added a commit that referenced this pull request Sep 3, 2023
* Wait for resource exists before sampling for condition

* return None in self.instance

* TimeoutSampler: rempve dulicate exception log

Co-authored-by: Meni Yakove <441263+myakove@users.noreply.github.com>
myakove added a commit that referenced this pull request Sep 3, 2023
#1404)

* Wait for resource exists before sampling for condition

* return None in self.instance

* TimeoutSampler: rempve dulicate exception log

Co-authored-by: redhat-qe-bot <108792105+redhat-qe-bot@users.noreply.github.com>
myakove added a commit that referenced this pull request Aug 11, 2025
* Wait for resource exists before sampling for condition

* return None in self.instance

* TimeoutSampler: rempve dulicate exception log
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants