Skip to content
This repository has been archived by the owner on Apr 20, 2021. It is now read-only.

[ios_l3_interfaces] with supported operations #18

Open
wants to merge 36 commits into
base: devel
Choose a base branch
from

Conversation

justjais
Copy link

No description provided.

@ansible-zuul
Copy link
Contributor

ansible-zuul bot commented May 29, 2019

Build failed.

@ansible-zuul
Copy link
Contributor

ansible-zuul bot commented May 29, 2019

Build failed.

@justjais justjais marked this pull request as ready for review June 5, 2019 14:41
@ansible-zuul
Copy link
Contributor

ansible-zuul bot commented Jun 5, 2019

Build failed.

@justjais justjais changed the title Draft for IOS L3 interface<WIP> IOS L3 interface with supported operations Jun 6, 2019
@ansible-zuul
Copy link
Contributor

ansible-zuul bot commented Jun 6, 2019

Build failed.

@ansible-zuul
Copy link
Contributor

ansible-zuul bot commented Jun 6, 2019

Build failed.

@ansible-zuul
Copy link
Contributor

ansible-zuul bot commented Jun 18, 2019

Build failed.

@ikhan2010
Copy link
Collaborator

@justjais can you please fix the linting issues?

@ansible-zuul
Copy link
Contributor

ansible-zuul bot commented Jun 28, 2019

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.

Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
Signed-off-by: Sumit Jaiswal <[email protected]>
@ansible-zuul
Copy link
Contributor

ansible-zuul bot commented Jun 28, 2019

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.

Copy link

@NilashishC NilashishC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to assert before, after and list of commands generated. The ordering of the before/after list of dictionaries are not consistent across consecutive runs in different environments, which may result in integration test failures in Zuul (as observed with VyOS tests in Zuul). Since we don't have an identifier for the dicts generated, we need to convert them to a set and then assert (using symmetric_difference), thereby making sure that changes in ordering does not affect test results.

@ikhan2010 ikhan2010 changed the title IOS L3 interface with supported operations [ios_l3_interfaces] with supported operations Jul 15, 2019
@ansible-zuul
Copy link
Contributor

ansible-zuul bot commented Aug 12, 2019

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.

@ansible-zuul
Copy link
Contributor

ansible-zuul bot commented Aug 19, 2019

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants