Skip to content

[CI] compare with the calculated SHA commit#336

Merged
v1v merged 2 commits intoelastic:masterfrom
v1v:feature/fix-variables
Sep 3, 2020
Merged

[CI] compare with the calculated SHA commit#336
v1v merged 2 commits intoelastic:masterfrom
v1v:feature/fix-variables

Conversation

@v1v
Copy link
Copy Markdown
Member

@v1v v1v commented Sep 3, 2020

Otherwise we can see some errors like:

09:58:21  + git diff --name-only 56d18114fc033189fd9ea05309d088ef8eadca00...061d4bc14c337e5716b0bb92ee5a3c19dd250937
09:58:21  fatal: Invalid symmetric difference expression 56d18114fc033189fd9ea05309d088ef8eadca00...061d4bc14c337e5716b0bb92ee5a3c19dd250937

56d1811 is the sha commit from 56d1811

but 061d4bc14c337e5716b0bb92ee5a3c19dd250937 is the sha commit generated by the default checkout , but no the gitCheckout

09:57:59  Merging remotes/origin/master commit d32077ad5bcc847ddc9b4f9d3f13f04624a6decc into PR head commit 73bc50083f36841ce7a1ab746688284728552337
09:57:59  Merge succeeded, producing 061d4bc14c337e5716b0bb92ee5a3c19dd250937
09:57:59  Checking out Revision 061d4bc14c337e5716b0bb92ee5a3c19dd250937 (PR-333)

@v1v v1v self-assigned this Sep 3, 2020
@v1v v1v added the automation label Sep 3, 2020
@v1v v1v marked this pull request as ready for review September 3, 2020 09:48
@ghost
Copy link
Copy Markdown

ghost commented Sep 3, 2020

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Pull request #336 updated]

  • Start Time: 2020-09-03T09:51:42.466+0000

  • Duration: 3 min 34 sec

@v1v v1v requested a review from a team September 3, 2020 09:54
@v1v v1v merged commit c51d008 into elastic:master Sep 3, 2020
@v1v v1v deleted the feature/fix-variables branch September 3, 2020 10:06
v1v added a commit to felixbarny/apm that referenced this pull request Sep 3, 2020
…e_urls

* upstream/master:
  [CI] compare with the calculated SHA commit (elastic#336)
  add link to PHP documentation
  Link to create-agent-issues.sh in spec process
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants