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

Fix for binary upload step, use bash shell rather then default sh #86382

Closed
wants to merge 1 commit into from

Conversation

atalman
Copy link
Contributor

@atalman atalman commented Oct 6, 2022

This fixes the issue during upload:

Run # reference ends with an RC suffix
  # reference ends with an RC suffix
  if [[ ${GITHUB_REF_NAME} = *-rc[0-9]* ]]; then
    echo "UPLOAD_CHANNEL=test" >> "$GITHUB_ENV"
  fi
  shell: sh -e {0}
/__w/_temp/f045f5d8-ddb.sh: 2: [[: not found

Test failure:
https://github.com/pytorch/pytorch/actions/runs/3199561387/jobs/5225448559

Test success:
https://github.com/pytorch/pytorch/actions/runs/3199573560/jobs/5225480345

Error started when we switched to: continuumio/miniconda3:4.12.0

@atalman atalman requested a review from a team as a code owner October 6, 2022 18:28
@pytorch-bot pytorch-bot bot added the topic: not user facing topic category label Oct 6, 2022
@pytorch-bot
Copy link

pytorch-bot bot commented Oct 6, 2022

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/86382

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit 1bfd24e:
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@atalman atalman added the ciflow/binaries_conda Trigger binary build and upload jobs for conda on the PR label Oct 6, 2022
Test

Fix for the binary upload
@atalman atalman changed the title Testing Fix for binary upload step Oct 6, 2022
@atalman atalman changed the title Fix for binary upload step Fix for binary upload step, use bash shell rather then default sh Oct 6, 2022
Copy link
Contributor

@weiwangmeta weiwangmeta left a comment

Choose a reason for hiding this comment

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

Good catch!

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Oct 6, 2022
@atalman
Copy link
Contributor Author

atalman commented Oct 6, 2022

@pytorchmergebot merge -f "This is CI binaries nightly change"

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

facebook-github-bot pushed a commit that referenced this pull request Oct 7, 2022
…6382) (#86382)

Summary:
This fixes the issue during upload:

```
Run # reference ends with an RC suffix
  # reference ends with an RC suffix
  if [[ ${GITHUB_REF_NAME} = *-rc[0-9]* ]]; then
    echo "UPLOAD_CHANNEL=test" >> "$GITHUB_ENV"
  fi
  shell: sh -e {0}
/__w/_temp/f045f5d8-ddb.sh: 2: [[: not found
```

Test failure:
https://github.com/pytorch/pytorch/actions/runs/3199561387/jobs/5225448559

Test success:
https://github.com/pytorch/pytorch/actions/runs/3199573560/jobs/5225480345

Error started when we switched to: continuumio/miniconda3:4.12.0

Pull Request resolved: #86382
Approved by: https://github.com/weiwangmeta

Test Plan: contbuild & OSS CI, see https://hud.pytorch.org/commit/pytorch/pytorch/bf746798841bd42c9b849716f9eeefde3271e93d

Reviewed By: seemethere

Differential Revision: D40167171

Pulled By: seemethere

fbshipit-source-id: d18689b94b06d02dd12c6159b29df54de9e3699e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ciflow/binaries_conda Trigger binary build and upload jobs for conda on the PR ciflow/trunk Trigger trunk jobs on your pull request cla signed Merged topic: not user facing topic category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants