Skip to content

Instantly share code, notes, and snippets.

@4383
Created July 31, 2020 14:26
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save 4383/14c7c46b16bf79f127c0d0ac557c6b71 to your computer and use it in GitHub Desktop.
Save 4383/14c7c46b16bf79f127c0d0ac557c6b71 to your computer and use it in GitHub Desktop.
oslo-cookiecutter
│13:22:49 hberaud | smcginnis, fungi: o/ any idea what's the problem here or how to identify him? => https://review.opendev.org/#/c/743972/1
│13:23:39 hberaud | CI logs are not really verbose so we can't find the issue, and locally everything works fine.
│14:23:46 smcginnis | hberaud: tox isn't in the default PATH, so https://opendev.org/openstack/oslo-cookiecutter/src/branch/master/tools/test_tox_targets.sh#L33
│ | fails to find it.
│14:24:11 smcginnis | I think that script needs to be updated to take the environment variable that gets set, or else the zuul configuration needs to add tox to
│ | the PATH.
│14:24:57 smcginnis | The tox command ends up being /home/zuul/.local/tox/bin/tox
│14:25:54 smcginnis | There's a job variable {{ tox_executable }} that points to that.
│14:26:14 smcginnis | I _thought_ it was also set in an environment variable, but not so sure now.
│14:26:30 smcginnis | Somehow we need to get that passed through so the script can use the right tox binary.
│14:40:56 hberaud | smcginnis: oh thanks, excellent
│16:24:14 fungi | hberaud: smcginnis: also the ensure-tox role *can* make the tox executable appear in the default execution path:
│ | https://zuul-ci.org/docs/zuul-jobs/python-roles.html#rolevar-ensure-tox.ensure_global_symlinks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment