search endpoint failure correction#5280
search endpoint failure correction#5280openshift-merge-robot merged 1 commit intocontainers:masterfrom
Conversation
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: baude, mheon The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
openshift-ci-robot
added
the
approved
|
/hold Cancel hold once tests pass. |
openshift-ci-robot
added
do-not-merge/hold
|
LGTM but the tests are fighting you. |
|
Same thing: #5282. Needs rebase against current master. I'm sorry, this was just really rotten timing: two simultaneous PRs that just happened to land within hours of each other without a git conflict but causing a real test conflict. |
when returning an invalid search, a return was omitted triggering a null on the consumer end. Fixes: containers#5228 Signed-off-by: Brent Baude <[email protected]>
|
rebased |
|
Restarting failed tests |
|
ARGH. It's the fedora registry flake which I thought I had fixed. I don't see my "sleeping & retrying" messages and am really confused. I've restarted the two fedora-registry flakes and am looking into the third ( |
|
/lgtm |
openshift-ci-robot
removed
the
do-not-merge/hold
github-actions
bot
added
the
locked - please file new issue/PR
7 participants
when returning an invalid search, a return was omitted triggering a null on the consumer end.
Fixes: #5228
Signed-off-by: Brent Baude [email protected]