Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

To fix the bug where description was being parsed incorrectly as Interface in of eos_*_interfaces #140

Merged
merged 2 commits into from
Jan 7, 2021

Conversation

justjais
Copy link
Member

@justjais justjais commented Jan 7, 2021

SUMMARY

To fix the bug where the description was being parsed incorrectly as Interface in of eos_*_interfaces. Fixes #86

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

eos_*_interfaces

ADDITIONAL INFORMATION

@justjais justjais requested a review from GomathiselviS January 7, 2021 11:26
@justjais justjais added this to the January'21 milestone Jan 7, 2021
@justjais justjais added the gate Gate PR in Zuul CI label Jan 7, 2021
@ansible-zuul ansible-zuul bot merged commit ce92477 into ansible-collections:main Jan 7, 2021
GomathiselviS added a commit to GomathiselviS/eos that referenced this pull request Jan 21, 2021
This reverts commit bfe00bf.

Revert "Revert "Change short desction (ansible-collections#123)""

This reverts commit 90c7d21.

eos_interfaces: Fix issue with 'forced' speed configuration. (ansible-collections#120)

eos_interfaces: Fix issue with 'forced' speed configuration.

Reviewed-by: https://github.com/apps/ansible-zuul

Add documentation to use the l2_interfaces module for L2 configs, in l3_interfaces module. (ansible-collections#102)

Add documentation to use the l2_interfaces module for L2 configs, in l3_interfaces module.

Reviewed-by: https://github.com/apps/ansible-zuul

Remove local connection testing for integration jobs (ansible-collections#127)

Remove local connection testing for integration jobs

Reviewed-by: https://github.com/apps/ansible-zuul

Drop ansible from requirement.txt (ansible-collections#128)

Drop ansible from requirement.txt

Reviewed-by: https://github.com/apps/ansible-zuul

Add eos_ospf_interfaces resource module (ansible-collections#125)

Add eos_ospf_interfaces resource module

Reviewed-by: https://github.com/apps/ansible-zuul

Release prep version 1.2.0 (ansible-collections#129)

Release prep version 1.2.0

Reviewed-by: https://github.com/apps/ansible-zuul

Add version key to galaxy.yaml (ansible-collections#132)

Add version key to galaxy.yaml

Reviewed-by: https://github.com/apps/ansible-zuul

To fix the bug where description was being parsed incorrectly as Interface in of eos_*_interfaces (ansible-collections#140)

To fix the bug where description was being parsed incorrectly as Interface in of eos_*_interfaces

Reviewed-by: https://github.com/apps/ansible-zuul

Fix list.copy()  and idempotent issues (replaced and overridden) (ansible-collections#142)

Fix list.copy()  and idempotent issues (replaced and overridden)

Reviewed-by: https://github.com/apps/ansible-zuul

Update default values in module argspec and docs

Signed-off-by: GomathiselviS <[email protected]>

linters fix
GomathiselviS added a commit to GomathiselviS/eos that referenced this pull request Jan 22, 2021
This reverts commit bfe00bf.

Revert "Revert "Change short desction (ansible-collections#123)""

This reverts commit 90c7d21.

eos_interfaces: Fix issue with 'forced' speed configuration. (ansible-collections#120)

eos_interfaces: Fix issue with 'forced' speed configuration.

Reviewed-by: https://github.com/apps/ansible-zuul

Add documentation to use the l2_interfaces module for L2 configs, in l3_interfaces module. (ansible-collections#102)

Add documentation to use the l2_interfaces module for L2 configs, in l3_interfaces module.

Reviewed-by: https://github.com/apps/ansible-zuul

Remove local connection testing for integration jobs (ansible-collections#127)

Remove local connection testing for integration jobs

Reviewed-by: https://github.com/apps/ansible-zuul

Drop ansible from requirement.txt (ansible-collections#128)

Drop ansible from requirement.txt

Reviewed-by: https://github.com/apps/ansible-zuul

Add eos_ospf_interfaces resource module (ansible-collections#125)

Add eos_ospf_interfaces resource module

Reviewed-by: https://github.com/apps/ansible-zuul

Release prep version 1.2.0 (ansible-collections#129)

Release prep version 1.2.0

Reviewed-by: https://github.com/apps/ansible-zuul

Add version key to galaxy.yaml (ansible-collections#132)

Add version key to galaxy.yaml

Reviewed-by: https://github.com/apps/ansible-zuul

To fix the bug where description was being parsed incorrectly as Interface in of eos_*_interfaces (ansible-collections#140)

To fix the bug where description was being parsed incorrectly as Interface in of eos_*_interfaces

Reviewed-by: https://github.com/apps/ansible-zuul

Fix list.copy()  and idempotent issues (replaced and overridden) (ansible-collections#142)

Fix list.copy()  and idempotent issues (replaced and overridden)

Reviewed-by: https://github.com/apps/ansible-zuul

update docs

Signed-off-by: GomathiselviS <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gate Gate PR in Zuul CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

All of eos_*_interfaces module parse interface incorrectly if the description has an interface keyword in it
2 participants