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

Refactor(eos_cli_config_gen): Wildcard dict to list for ip_extcommunity_lists #1602

Conversation

KonikaChaurasiya-GSLab
Copy link
Contributor

@KonikaChaurasiya-GSLab KonikaChaurasiya-GSLab commented Mar 17, 2022

"Wildcard Keyed Dict" to "List of Dicts" Data model migration

Data model key

ip_extcommunity_lists

Checklist

Contributor Checklist

  • Update README_v4.0.md with new data model
  • Update all host_vars under molecule scenario eos_cli_config_gen_v4.0 with new data model
  • Update templates/eos/< template >.j2 and templates/documentation/< template >.j2:
    • Add arista.avd.convert_dicts('<primary key>') filter for loops previously using wildcard keys
    • Update arista.avd.natural_sort('<primary key>') to sort on the primary key (if applicable)
  • Run molecule cd ansible_collections/arista/avd/molecule ; make cli-4.0-schema
    • Verify no changes to generated configs/docs

Reviewer Checklist

  • Reviewer 1: Claus

    • Verify data model changes
    • Verify that all molecule host_vars under eos_cli_config_gen_v4.0 has been updated to the new data model
    • Verify no changes to configs/docs on any molecule scenario
    • Verify that CI pass
  • Reviewer 2: Tony

    • Verify data model changes
    • Verify that all molecule host_vars under eos_cli_config_gen_v4.0 has been updated to the new data model
    • Verify no changes to configs/docs on any molecule scenario
    • Verify that CI pass

@KonikaChaurasiya-GSLab KonikaChaurasiya-GSLab force-pushed the eos_cli_config_gen/dict2list/ip-extended-community-lists branch from 57cff11 to 9913c6f Compare March 17, 2022 13:24
@github-actions github-actions bot added the state: CI Updated CI scenario have been updated in the PR label Mar 17, 2022
@ankudinov ankudinov self-requested a review March 31, 2022 09:19
@carlbuchmann carlbuchmann modified the milestone: v3.4.0 Apr 1, 2022
@KonikaChaurasiya-GSLab KonikaChaurasiya-GSLab marked this pull request as ready for review April 21, 2022 05:55
@github-actions
Copy link

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

@github-actions github-actions bot added the state: conflict PR with conflict label Apr 21, 2022
@github-actions
Copy link

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

@github-actions github-actions bot removed the state: conflict PR with conflict label Apr 22, 2022
…ty_lists after convertDict filter code change
Copy link
Contributor

@ClausHolbechArista ClausHolbechArista left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@tgodaA tgodaA left a comment

Choose a reason for hiding this comment

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

LGTM

@ClausHolbechArista ClausHolbechArista merged commit 9d630b4 into aristanetworks:devel Apr 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
EPIC - AVD Schema role: eos_cli_config_gen issue related to eos_cli_config_gen role state: CI Updated CI scenario have been updated in the PR state: Documentation role Updated
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants