Aleksandr Kurbatov authored 4 months ago and Karel van Klink committed 4 months ago

Using `add_pe_to_p` will check that all PE neighbors are present in the
list of BGP neighbors in P-ONLY group.
`verify_p_ibgp` had a custom template to regenerate the entire BGP
config on a P router - this is legacy.

 Conflicts:
	gso/annotations/__init__.py
	gso/annotations/base_site.py
	gso/annotations/coordinates.py
	gso/annotations/country_code.py
	gso/annotations/interfaces.py
	gso/annotations/ip_address.py
	gso/annotations/netbox_router.py
	gso/annotations/site_name.py
	gso/annotations/snmp.py
	gso/annotations/tt_number.py
	gso/annotations/unique_field.py
	gso/cli/imports.py
	gso/products/product_blocks/iptrunk.py
	gso/products/product_blocks/lan_switch_interconnect.py
	gso/products/product_blocks/site.py
	gso/services/librenms_client.py
	gso/types/__init__.py
	gso/types/base_site.py
	gso/types/coordinates.py
	gso/types/country_code.py
	gso/types/interfaces.py
	gso/types/netbox_router.py
	gso/types/site_name.py
	gso/types/snmp.py
	gso/types/tt_number.py
	gso/types/unique_field.py
	gso/utils/helpers.py
	gso/utils/types/__init__.py
	gso/utils/types/base_site.py
	gso/utils/types/coordinates.py
	gso/utils/types/country_code.py
	gso/utils/types/interfaces.py
	gso/utils/types/netbox_router.py
	gso/utils/types/site_name.py
	gso/utils/types/snmp.py
	gso/utils/types/tt_number.py
	gso/utils/types/unique_field.py
	gso/workflows/iptrunk/deploy_twamp.py
	gso/workflows/iptrunk/modify_isis_metric.py
	gso/workflows/router/redeploy_base_config.py
	gso/workflows/site/create_imported_site.py
	gso/workflows/site/create_site.py
	gso/workflows/site/modify_site.py
	test/conftest.py
	test/schemas/test_types.py
	test/services/test_librenms_client.py
	test/workflows/iptrunk/test_create_imported_iptrunk.py

Karel van Klink authored 6 months ago and Mohammad Torkashvand committed 6 months ago

Automatically skip a playbook execution if the inventory is empty to prevent hangs

inventory: need to exclude this router from inventory generator, because
at this point in WF it is still `P` role.

extra_vars: need to exclude this promoted router, since it is still `P`
role.

Changes:
- sequence of steps
- small changes in steps extra_vars, inventories, verbs, descriptions