Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for Test Conda Token #4530

Merged
merged 1 commit into from
Aug 29, 2023
Merged

Conversation

osalpekar
Copy link
Member

@osalpekar osalpekar commented Aug 29, 2023

We will use a separate CONDA_PYTORCHBOT_TOKEN_TEST for using in a test environment

Fixes #4529

TODO:

  1. Add the token to the other domain repos
  2. Add to audio caller: Separate Test Token for Conda Uploads audio#3582
  3. Add to vision caller: Separate Test Token for Conda Uploads vision#7900

@osalpekar osalpekar requested a review from atalman August 29, 2023 20:02
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 29, 2023
Copy link
Contributor

@atalman atalman left a comment

Choose a reason for hiding this comment

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

LGTM

@vercel
Copy link

vercel bot commented Aug 29, 2023

@osalpekar is attempting to deploy a commit to the Meta Open Source Team on Vercel.

A member of the Team first needs to authorize it.

@osalpekar osalpekar merged commit b6afaf4 into pytorch:main Aug 29, 2023
27 of 31 checks passed
pytorchmergebot pushed a commit to pytorch/pytorch that referenced this pull request Aug 30, 2023
There are more issues that I expect at the beginning:

* Triton was uploaded on `main` instead of `nightly` and release branch
* The environment `conda-aws-upload` wasn't used correctly in both wheel and conda upload
* Conda update wasn't run in a separate ephemeral runner
* Duplicated upload logic, should have just use `bash .circleci/scripts/binary_upload.sh` instead
* Handle `CONDA_PYTORCHBOT_TOKEN` and `CONDA_PYTORCHBOT_TOKEN_TEST` tokens in a similar way as pytorch/test-infra#4530

Part of #108154
Pull Request resolved: #108187
Approved by: https://github.com/atalman
huydhn added a commit to huydhn/pytorch that referenced this pull request Aug 30, 2023
There are more issues that I expect at the beginning:

* Triton was uploaded on `main` instead of `nightly` and release branch
* The environment `conda-aws-upload` wasn't used correctly in both wheel and conda upload
* Conda update wasn't run in a separate ephemeral runner
* Duplicated upload logic, should have just use `bash .circleci/scripts/binary_upload.sh` instead
* Handle `CONDA_PYTORCHBOT_TOKEN` and `CONDA_PYTORCHBOT_TOKEN_TEST` tokens in a similar way as pytorch/test-infra#4530

Part of pytorch#108154
osalpekar added a commit that referenced this pull request Aug 30, 2023
Cherry-pick of #4530 onto
release/2.1 branch
atalman pushed a commit to pytorch/pytorch that referenced this pull request Aug 30, 2023
There are more issues that I expect at the beginning:

* Triton was uploaded on `main` instead of `nightly` and release branch
* The environment `conda-aws-upload` wasn't used correctly in both wheel and conda upload
* Conda update wasn't run in a separate ephemeral runner
* Duplicated upload logic, should have just use `bash .circleci/scripts/binary_upload.sh` instead
* Handle `CONDA_PYTORCHBOT_TOKEN` and `CONDA_PYTORCHBOT_TOKEN_TEST` tokens in a similar way as pytorch/test-infra#4530

Part of #108154
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Automate CONDA_PYTORCHBOT_TOKEN_TEST setting for core and domains
5 participants