Skip to content
This repository has been archived by the owner on Nov 5, 2019. It is now read-only.

Foo #175

Closed
wants to merge 1 commit into from
Closed

Foo #175

wants to merge 1 commit into from

Conversation

dhermes
Copy link
Contributor

@dhermes dhermes commented May 13, 2015

@nathanielmanistaatgoogle @craigcitro This is to test which environment variables are actually set (without printing their values).

I already am tracking this in googleapis/google-cloud-python#874 and get

$ python -c "import os; print '\n'.join(os.environ.keys())"
_
SHLVL
VIRTUAL_ENV
PWD
PYTHONHASHSEED
PATH

even though the Travis docs say all of

TRAVIS=true
TRAVIS_BRANCH
TRAVIS_BUILD_DIR
TRAVIS_BUILD_ID
TRAVIS_BUILD_NUMBER
TRAVIS_COMMIT
TRAVIS_COMMIT_RANGE
TRAVIS_JOB_ID
TRAVIS_JOB_NUMBER
TRAVIS_PULL_REQUEST
TRAVIS_SECURE_ENV_VARS
TRAVIS_REPO_SLUG
TRAVIS_OS_NAME
TRAVIS_TAG

should be set.

@craigcitro
Copy link
Contributor

have you looked at their twitter account to see if this is a known issue?

i can hop in the slack channel shortly to ask if need be ...

@dhermes
Copy link
Contributor Author

dhermes commented May 13, 2015

No mention of it on Twitter. I also did a search for "travis environment variables" and searched https://github.com/travis-ci/travis-ci/issues

@dhermes
Copy link
Contributor Author

dhermes commented May 13, 2015

It seems @tseaver has found the issue. It's tox.

@dhermes dhermes closed this May 13, 2015
@dhermes dhermes deleted the env-var-fail branch May 13, 2015 18:40
@tseaver
Copy link

tseaver commented May 13, 2015

tox.ini needs a passenv = element in the relevant testenv stanzas, similar to googleapis/google-cloud-python#875

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants