dsp-jetpack / JetPack

Apache License 2.0
30 stars 64 forks source link

Updated powerflex templates for all the Core NIC Configurations #655

Closed usama2490 closed 3 years ago

usama2490 commented 3 years ago

For example, to enable powerflex, there would be a small change need to make in the file path, like that in INI.. "nic_env_file=4_port/nic_environment_powerflex.yaml"

@gaelrehault kindly review the PR

Regards,

gaelrehault commented 3 years ago

Could you please update the templates to use the rhel8 interface names ?

usama2490 commented 3 years ago

Could you please update the templates to use the rhel8 interface names ?

Yeah I can do that gael. But interface names are overwritten in these templates with the values defined in the INI file while the deployment. Existing templates have also the older names for interfaces, do you want me to update them also for the record or just the one defined in the new flex files?

gaelrehault commented 3 years ago

Let s update all of them please .

usama2490 commented 3 years ago

Yeah sure gael, i am on it.

usama2490 commented 3 years ago

I have updated the interface names and did some additional changes in validation checks in the case of xsp and csp profiles. We have successfully deployed powerflex with 5_port/nic_environment_powerflex.yaml on R87, so configurations for all the other nfv features should be good.

PR is good to go.