Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

Fix bugs found in integration test #3214

Merged
merged 3 commits into from
Dec 21, 2020
Merged

Fix bugs found in integration test #3214

merged 3 commits into from
Dec 21, 2020

Conversation

liuzhe-lz
Copy link
Contributor

No description provided.

@@ -15,36 +15,33 @@ jobs:
echo "##vso[task.setvariable variable=PATH]${PATH}:${HOME}/.local/bin"
echo "##vso[task.setvariable variable=NNI_RELEASE]999.$(date -u +%Y%m%d%H%M%S)"

python3 -m pip install -U --upgrade pip setuptools
python3 -m pip install -U pytest
python3 -m pip install --upgrade pip setuptools
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why remove -U?

Copy link
Contributor Author

@liuzhe-lz liuzhe-lz Dec 21, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because it's duplicate to --upgrade.
I mistakenly thought it was --user.

displayName: Install customized tuner

- script: |
set -e
docker login -u nnidev -p $(docker_hub_password)
echo '## Build docker image ##'
docker build --build-arg NNI_RELEASE=${NNI_RELEASE} -t nnidev/nni-it-pai:latest .
docker build --build-arg NNI_RELEASE=${NNI_RELEASE} -t nnidev/nni-nightly .
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nightly?

Copy link
Contributor Author

@liuzhe-lz liuzhe-lz Dec 21, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Each day scheduled pipelines will build and push several docker images with same content. So it's effectively the nightly version.
I'm planning to write a pipeline to trigger all IT pipelines (after this release), which will build and push docker image at first. This is a preparation for that.

@liuzhe-lz liuzhe-lz merged commit e020814 into master Dec 21, 2020
@liuzhe-lz liuzhe-lz deleted the dev-it2 branch December 21, 2020 03:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants