Dell OS10 needs OSPF process ID configured before BGP in case of import policy

[jbemmel]

Working on adding support for BGP route imports, I encountered an issue where test 31-import-policy fails because OS10 requires the OSPF process id to be configured before adding redistribute ospf X (where X is the OSPF process ID)

Can we add a global constraint to configure OSPF before BGP, or would you prefer a quirk which reorders the modules only for Dell OS10?

[ipspace]

I encountered an issue where test 31-import-policy fails because OS10 requires the OSPF process id to be configured before adding redistribute ospf X (where X is the OSPF process ID)

You have the infinite bad luck of picking the crappiest operating systems out there :(

However, how do you want to handle two-way redistribution between two routing protocols where you have to specify instance ID?

Can we add a global constraint to configure OSPF before BGP

If anything, it would make sense to configure all IGPs before BGP. I created a PR that does that and am currently running tests to see whether anything crashes. Expect some results in a day or two...

or would you prefer a quirk which reorders the modules only for Dell OS10?

That won't work for complex reasons that have to do with the crappy way Ansible handles loops. The configuration sequence of modules is set globally and the node "module" list is only consulted to see whether to include that task list or not.

[jbemmel]

You have the infinite bad luck of picking the crappiest operating systems out there :(

I didn't get to pick, it's inherited

However, how do you want to handle two-way redistribution between two routing protocols where you have to specify instance ID?

Yeah, when that moment comes we may have to add router ospf X to the BGP template (ugh). For now, thanks for changing the global order - it's good to have a defined consistent order independent of whatever the user provided

[ipspace]

For now, thanks for changing the global order - it's good to have a defined consistent order independent of whatever the user provided

We're still not there yet. Some test might still fail...