Skip to content

Conversation

@r-barnes
Copy link
Contributor

Test Plan: Sandcastle

Differential Revision: D35194935

@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Mar 29, 2022

🔗 Helpful links

💊 CI failures summary and remediations

As of commit bcf41f7 (more details on the Dr. CI page):


  • 1/1 failures introduced in this PR

🕵️ 1 new failure recognized by patterns

The following CI failures do not appear to be due to upstream breakages:

See GitHub Actions build pull / linux-bionic-rocm4.5-py3.7 / build (1/1)

Step: "Build" (full log | diagnosis details | 🔁 rerun)

2022-03-30T00:47:24.5162360Z �[36;1m echo "ERR...t available for the merge-base of your branch"�[0m
2022-03-30T00:47:24.5159511Z �[36;1mfi�[0m
2022-03-30T00:47:24.5159740Z �[36;1m# Covers the case where a previous tag doesn't exist for the tree�[0m
2022-03-30T00:47:24.5160075Z �[36;1m# this is only really applicable on trees that don't have `.circleci/docker` at its merge base, i.e. nightly�[0m
2022-03-30T00:47:24.5160388Z �[36;1mif ! git rev-parse "$MERGE_BASE:.circleci/docker"; then�[0m
2022-03-30T00:47:24.5160728Z �[36;1m  echo "Directory '.circleci/docker' not found in commit $MERGE_BASE, you should probably rebase onto a more recent commit"�[0m
2022-03-30T00:47:24.5161008Z �[36;1m  exit 1�[0m
2022-03-30T00:47:24.5161174Z �[36;1mfi�[0m
2022-03-30T00:47:24.5161398Z �[36;1mPREVIOUS_DOCKER_TAG=$(git rev-parse "$MERGE_BASE:.circleci/docker")�[0m
2022-03-30T00:47:24.5161729Z �[36;1m# If no image exists but the hash is the same as the previous hash then we should error out here�[0m
2022-03-30T00:47:24.5162030Z �[36;1mif [[ "${PREVIOUS_DOCKER_TAG}" = "${DOCKER_TAG}" ]]; then�[0m
2022-03-30T00:47:24.5162360Z �[36;1m  echo "ERROR: Something has gone wrong and the previous image isn't available for the merge-base of your branch"�[0m
2022-03-30T00:47:24.5162689Z �[36;1m  echo "       contact the PyTorch team to restore the original images"�[0m
2022-03-30T00:47:24.5162915Z �[36;1m  exit 1�[0m
2022-03-30T00:47:24.5163128Z �[36;1mfi�[0m
2022-03-30T00:47:24.5163314Z �[36;1mecho ::set-output name=rebuild::yes�[0m
2022-03-30T00:47:24.5173859Z shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
2022-03-30T00:47:24.5174074Z env:
2022-03-30T00:47:24.5174220Z   IN_CI: 1
2022-03-30T00:47:24.5174378Z   IS_GHA: 1
2022-03-30T00:47:24.5174589Z   BASE_REVISION: 9cc848c3f86271f1d90c72888147224091ee6d7a
2022-03-30T00:47:24.5174975Z   DOCKER_IMAGE: 308535385114.dkr.ecr.us-east-1.amazonaws.com/pytorch/pytorch-linux-bionic-rocm4.5-py3.7:caae90dbc9b94ddfa41f21ceba5aa63bb73efc2a

This comment was automatically generated by Dr. CI (expand for details).

Please report bugs/suggestions to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

@facebook-github-bot facebook-github-bot added oncall: jit Add this issue/PR to JIT oncall triage queue fb-exported labels Mar 29, 2022
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D35194935

Summary: Pull Request resolved: pytorch#74923

Test Plan: Sandcastle

Differential Revision: D35194935

fbshipit-source-id: 7fc8931c2cc76a3ed5b467a7028af6a653c8cf97
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D35194935

facebook-github-bot pushed a commit that referenced this pull request Mar 31, 2022
Summary: Pull Request resolved: #74923

Test Plan: Sandcastle

Reviewed By: malfet

Differential Revision: D35194935

fbshipit-source-id: f5ec5be87cdf775eb9c99f8c3baed6b0366dda49
pytorchmergebot pushed a commit that referenced this pull request Mar 31, 2022
Summary: Pull Request resolved: #74923

Test Plan: Sandcastle

Reviewed By: malfet

Differential Revision: D35194935

fbshipit-source-id: f5ec5be87cdf775eb9c99f8c3baed6b0366dda49
(cherry picked from commit 7284c4e)
@github-actions
Copy link
Contributor

Hey @r-barnes.
You've committed this PR, but it does not have both a 'release notes: ...' and 'topics: ...' label. Please add one of each to the PR. The 'release notes: ...' label should represent the part of PyTorch that this PR changes (fx, autograd, distributed, etc) and the 'topics: ...' label should represent the kind of PR it is (not user facing, new feature, bug fix, perf improvement, etc). The list of valid labels can be found here for the 'release notes: ...' and here for the 'topics: ...'.
For changes that are 'topic: not user facing' there is no need for a release notes label.

@malfet
Copy link
Contributor

malfet commented Mar 31, 2022

Reverting, as this PR broke rocm builds (as clearly seen in PR CI): https://github.com/pytorch/pytorch/runs/5765634148?check_suite_focus=true

@facebook-github-bot
Copy link
Contributor

This pull request has been reverted by 10677f85358dc82fbcd7638d2bc3c6545aa16f01. To re-land this change, please open another pull request, assignthe same reviewers, fix the CI failures that caused the revert and make sure that the failing CI runs on the PR by applying the proper ciflow label (e.g., ciflow/trunk).

@facebook-github-bot
Copy link
Contributor

This pull request has been reverted by c40a009. To re-land this change, please open another pull request, assignthe same reviewers, fix the CI failures that caused the revert and make sure that the failing CI runs on the PR by applying the proper ciflow label (e.g., ciflow/trunk).

1 similar comment
@facebook-github-bot
Copy link
Contributor

This pull request has been reverted by c40a009. To re-land this change, please open another pull request, assignthe same reviewers, fix the CI failures that caused the revert and make sure that the failing CI runs on the PR by applying the proper ciflow label (e.g., ciflow/trunk).

jjsjann123 pushed a commit to jjsjann123/nvfuser that referenced this pull request Oct 29, 2022
Summary: Pull Request resolved: pytorch/pytorch#74923

Test Plan: Sandcastle

Reviewed By: malfet

Differential Revision: D35194935

fbshipit-source-id: f5ec5be87cdf775eb9c99f8c3baed6b0366dda49
(cherry picked from commit 7284c4ed7d57261d4936055e0c1a3f8f911fb1f0)
jjsjann123 pushed a commit to jjsjann123/nvfuser that referenced this pull request Nov 10, 2022
Summary: Pull Request resolved: pytorch/pytorch#74923

Test Plan: Sandcastle

Reviewed By: malfet

Differential Revision: D35194935

fbshipit-source-id: f5ec5be87cdf775eb9c99f8c3baed6b0366dda49
(cherry picked from commit 7284c4ed7d57261d4936055e0c1a3f8f911fb1f0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla signed fb-exported oncall: jit Add this issue/PR to JIT oncall triage queue Reverted

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants