Skip to content
This repository has been archived by the owner on Jan 10, 2023. It is now read-only.

Causes the correct server URL to be set in GitHub Actions. #441

Closed

Conversation

debayande
Copy link

@debayande debayande commented Jun 10, 2021

Signed-off-by: Debayan De debayande@users.noreply.github.com

Purpose

As a part of its routine for auto-detecting the underlying CI engine, the Bash uploader sets the build_url parameter to http://github.com/${GITHUB_REPOSITORY}/actions/runs/${GITHUB_RUN_ID} in L875 upon detecting GitHub Actions as the CI engine.

While this works fine for github.com, it breaks uploads with HTTP 500 errors when run on GitHub Enterprise, GitHub's self-hosted sibling.

This patch, if applied, will fix the aforementioned issue (which is filed as #440).

Notable Changes

No.

Tests and Risks?

This should be covered by existing tests.

Update the SHA hash files

Done.

@debayande debayande force-pushed the fix/use-correct-github-server-url branch 2 times, most recently from 731a331 to a50254e Compare June 10, 2021 14:49
@codecov
Copy link

codecov bot commented Jun 10, 2021

Codecov Report

Merging #441 (a5fc10f) into master (c3ccd59) will increase coverage by 0.22%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #441      +/-   ##
==========================================
+ Coverage   45.73%   45.95%   +0.22%     
==========================================
  Files          20       20              
  Lines        2858     2853       -5     
==========================================
+ Hits         1307     1311       +4     
+ Misses       1551     1542       -9     
Flag Coverage Δ
test 39.22% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
codecov 67.96% <100.00%> (+0.46%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c3ccd59...a5fc10f. Read the comment docs.

@debayande debayande force-pushed the fix/use-correct-github-server-url branch 2 times, most recently from eb10a97 to 8a3e0b3 Compare June 10, 2021 16:59
@debayande debayande force-pushed the fix/use-correct-github-server-url branch from 8a3e0b3 to f8530b1 Compare June 10, 2021 18:27
Signed-off-by: Debayan De <debayande@users.noreply.github.com>
Signed-off-by: Debayan De <debayande@users.noreply.github.com>
@debayande debayande force-pushed the fix/use-correct-github-server-url branch from f8530b1 to a5fc10f Compare June 10, 2021 18:43
@thomasrockhu
Copy link
Contributor

Superceded by #457

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants