podman icon indicating copy to clipboard operation
podman copied to clipboard

Fix several reliability problems with pasta DNS handling tests

Open dgibson opened this issue 1 year ago • 3 comments
trafficstars

I've had intermittent problems with the pasta DNS tests in podman. Here are some fixes that address at least some of them.

/cc @sbrivio-rh @Luap99

Does this PR introduce a user-facing change?

None

dgibson avatar Jul 19 '24 01:07 dgibson

Ephemeral COPR build failed. @containers/packit-build please check.

I guess we'll have to include dig in the images: dig has two advantages

dig has infinite advantages. I love dig. I deliberately choose not to include it in quay.io/testimage because of its dependencies: bind-tools requires a lot of useless-to-us stuff, and bloats testimage from 12MB to 19.

edsantiago avatar Jul 22 '24 15:07 edsantiago

@Luap99 @edsantiago where do we stand here on this one? seems like something we'd want to merge?

baude avatar Aug 12 '24 12:08 baude

Unless we get version that passes tests there doesn't seem to be anything for us to do so this is waiting on @dgibson.

Luap99 avatar Aug 12 '24 13:08 Luap99

A friendly reminder that this PR had no activity for 30 days.

github-actions[bot] avatar Sep 13 '24 00:09 github-actions[bot]

@Luap99 @edsantiago, finally got around to revising this, sorry it took so long.

A bunch of the Fedora 41 and rawhide tests are failing. I think it looks like an infrastructure issue unrelated to the actual changes.

dgibson avatar Sep 16 '24 07:09 dgibson

New version pushed. Only change is a rebase + an extra patch attempting to address @edsantiago's comment.

dgibson avatar Sep 18 '24 01:09 dgibson

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dgibson, Luap99

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

openshift-ci[bot] avatar Sep 23 '24 15:09 openshift-ci[bot]