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

Fix(eos_cli_config_gen): ip http client source interfaces cli not generated #3180

Conversation

carlbuchmann
Copy link
Member

@carlbuchmann carlbuchmann commented Sep 30, 2023

Change Summary

ip http client source interfaces cli not generated due to the incorrect if statement.
This only occurred when ip_http_client_source_interfaces was the only ip-client-source-interfaces key defined.

Component(s) name

arista.avd.eos_cli_config_gen

Proposed changes

Fix the typo in the if statement.

How to test

All molecule tests pass.

@carlbuchmann carlbuchmann requested review from a team as code owners September 30, 2023 01:01
@github-actions github-actions bot added the role: eos_cli_config_gen issue related to eos_cli_config_gen role label Sep 30, 2023
@carlbuchmann carlbuchmann force-pushed the fix-ip-http-client-sources-interfaces branch from 0c610a2 to fb86d90 Compare September 30, 2023 01:04
@carlbuchmann carlbuchmann added the type: bug Something isn't working label Sep 30, 2023
@carlbuchmann carlbuchmann added this to the v4.4.0 milestone Sep 30, 2023
@carlbuchmann carlbuchmann force-pushed the fix-ip-http-client-sources-interfaces branch from fb86d90 to 1f19faf Compare September 30, 2023 14:48
@github-actions github-actions bot added type: code quality CI and development toolset state: conflict PR with conflict labels Oct 2, 2023
@github-actions
Copy link

github-actions bot commented Oct 2, 2023

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@ClausHolbechArista
Copy link
Contributor

Please remove the last commit and try again.

@github-actions
Copy link

github-actions bot commented Oct 2, 2023

Conflicts have been resolved. A maintainer will review the pull request shortly.

@github-actions github-actions bot removed state: conflict PR with conflict type: code quality CI and development toolset labels Oct 2, 2023
@gmuloc
Copy link
Contributor

gmuloc commented Oct 2, 2023

Please remove the last commit and try again.

Fixed merged conflicts from Github UI

@ClausHolbechArista ClausHolbechArista requested review from a team October 2, 2023 12:46
@gmuloc gmuloc merged commit 1bac009 into aristanetworks:devel Oct 2, 2023
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rn: Fix(eos_cli_config_gen) role: eos_cli_config_gen issue related to eos_cli_config_gen role type: bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants