Openstack Mojo Testing

Code review comment for lp:~ajkavanagh/openstack-mojo-specs/fix-os-project-id-nova-creds

  1. fix-os-project-id-nova-creds
  2. Merge into mojo-openstack-specs-1709
Revision history for this message
Liam Young (gnuoy) wrote :

I think the issue may come from whether you have a novarc sourced when you run mojo. I'm +1 on removing the os.environ.get('OS_PROJECT_ID') from helper/utils/mojo_os_utils.py

Only issue for me seems to be some code duplication.

review: Needs Fixing

« Back to merge proposal