origin icon indicating copy to clipboard operation
origin copied to clipboard

Published 20 hours ago • verified publisher icon openshift

Add bond cni test

Open mmirecki opened this issue 2 years ago • 11 comments

Adding a test case for the bond-cni. The test checks if a pod with a bond is created successfully, and then checks connectivity between two pods with bond interfaces.

mmirecki avatar Sep 07 '22 10:09 mmirecki

/retest-required

mmirecki avatar Sep 08 '22 06:09 mmirecki

@dougbtv could you please take a look, thx

mmirecki avatar Sep 08 '22 09:09 mmirecki

/retest-required

mmirecki avatar Sep 08 '22 13:09 mmirecki

/retest-required

mmirecki avatar Sep 08 '22 17:09 mmirecki

/retest-required

mmirecki avatar Sep 09 '22 07:09 mmirecki

/assign @mfojtik

mmirecki avatar Sep 15 '22 10:09 mmirecki

@dougbtv Can we merge this?

mmirecki avatar Sep 19 '22 12:09 mmirecki

/retest-required

mmirecki avatar Sep 21 '22 14:09 mmirecki

/approve

dougbtv avatar Sep 21 '22 18:09 dougbtv

/assign @mfojtik

mmirecki avatar Sep 22 '22 07:09 mmirecki

@danwinship @dcbw Could you pleae take a look at this one?

mmirecki avatar Sep 29 '22 12:09 mmirecki

/retest-required

mmirecki avatar Oct 03 '22 07:10 mmirecki

/retest-required

mmirecki avatar Oct 03 '22 11:10 mmirecki

/retest-required

mmirecki avatar Oct 17 '22 10:10 mmirecki

/retest-required

mmirecki avatar Oct 18 '22 08:10 mmirecki

@danwinship @dougbtv Could you please take one more look?

mmirecki avatar Oct 18 '22 15:10 mmirecki

/retest-required

EinatGlottmann avatar Oct 20 '22 11:10 EinatGlottmann

@dcbw @knobunc @bparees Could you please take a look?

mmirecki avatar Oct 20 '22 21:10 mmirecki

/lgtm

dougbtv avatar Oct 20 '22 21:10 dougbtv

@smarterclayton @mfojtik @adambkaplan Could you please take a look at this?

mmirecki avatar Oct 21 '22 08:10 mmirecki

@danwinship could you please revisit this PR? I believe Marcin addressed your previous comments. Thank you!

cgoncalves avatar Oct 21 '22 09:10 cgoncalves

/approve

dougbtv avatar Oct 21 '22 12:10 dougbtv

/approve

knobunc avatar Oct 21 '22 12:10 knobunc

/retest-required

Remaining retests: 0 against base HEAD 6f17d51fd4b5a89e36c50b43ddad42e43f2d08ba and 2 for PR HEAD 786b57565196e776b756e7ebd50160678272121b in total

openshift-ci-robot avatar Oct 22 '22 22:10 openshift-ci-robot

/retest-required

cgoncalves avatar Oct 27 '22 07:10 cgoncalves

/retest-required

mmirecki avatar Oct 31 '22 11:10 mmirecki

/lgtm

cgoncalves avatar Nov 02 '22 10:11 cgoncalves

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cgoncalves, dougbtv, knobunc, maiqueb, mmirecki

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 Nov 02 '22 10:11 openshift-ci[bot]

/retest-required

mmirecki avatar Nov 02 '22 11:11 mmirecki

/retest-required

mmirecki avatar Nov 02 '22 14:11 mmirecki