Skip to content

Conversation

@aajisaka
Copy link
Member

@aajisaka aajisaka commented Mar 2, 2021

JIRA: https://issues.apache.org/jira/browse/YETUS-1102

  • Add --github-write-comment to back to the previous behavior.
  • Add back --github-use-emoji-vote option.

@aajisaka
Copy link
Member Author

aajisaka commented Mar 3, 2021

I'll look into this. -> #214 will fix this issue.

|  -1  |      docker  |   6m 27s   | Docker failed to build 
|      |              |            | yetus/yetus:b5a09398.

@aajisaka
Copy link
Member Author

aajisaka commented Mar 4, 2021

Re-ran all the job, and linkcheck is still failing:

  #44 ERROR: lstat /var/lib/docker/tmp/buildkit-mount087864928/root/go/bin: no such file or directory
  ------
   > importing cache manifest from apache/yetus-base:pull_213_merge:
  ------
  ------
   > importing cache manifest from apache/yetus-build:pull_213_merge:
  ------
  ------
   > importing cache manifest from apache/yetus:pull_213_merge:
  ------
  ------
   > [stage-10 24/25] COPY --from=checkmake /root/go/bin/* /usr/local/bin:
  ------
  lstat /var/lib/docker/tmp/buildkit-mount087864928/root/go/bin: no such file or directory
  Error: Process completed with exit code 1.

Probably the test referenced the old build cache.

@aajisaka
Copy link
Member Author

aajisaka commented Mar 4, 2021

GitHub Actions doesn't have "clear cache" feature. We have to change the cache key to clear the cache, and it is very frustrating.
actions/cache#2

@aajisaka
Copy link
Member Author

aajisaka commented Mar 4, 2021

After digging this issue, this issue is not related to the cache of GitHub Actions. However, I'm not sure why the old Dockerfile is used.
https://pipelines.actions.githubusercontent.com/4IRN7cmKQG30mFwpZCDL2WwRpIOhhBRpKKjsnasVdYJ4dd7vJI/_apis/pipelines/1/runs/793/signedlogcontent/3?urlExpires=2021-03-04T07%3A54%3A16.4608893Z&urlSigningMethod=HMACV1&urlSignature=B%2FSsQekc2fFD%2FI5O9vUFrVgS2ZMYrRwlqmcCw5kCjck%3D

2021-03-04T07:47:56.7163921Z #43 [checkmake 1/1] RUN go get -u github.com/mrtazz/checkmake
2021-03-04T07:47:56.7164901Z #43 sha256:a79f93a32d6630e29c4cb62f8fca54b97bebb0c0135d1e4b6e0bc81fca55b104
2021-03-04T07:47:56.7166093Z #43 2.648 go: downloading github.com/mrtazz/checkmake v0.0.0-20201029101907-575315c9924d
2021-03-04T07:47:56.7166602Z #43 DONE 4.0s

@aajisaka
Copy link
Member Author

aajisaka commented Mar 8, 2021

I tested in apache/hadoop#2745, and the comment looks good. I'll commit this tomorrow if there are no objections.

@aajisaka
Copy link
Member Author

aajisaka commented Mar 9, 2021

72 hours passed since the last commit. Merging this.

@aajisaka aajisaka merged commit 11eb9b0 into apache:main Mar 9, 2021
@aajisaka aajisaka deleted the YETUS-1102 branch March 9, 2021 05:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants