Skip to content

Use gh-pages to save performance benchmarks results#1469

Merged
codeboten merged 1 commit intoopen-telemetry:masterfrom
NathanielRN:benchmarks-on-gh-pages
Dec 13, 2020
Merged

Use gh-pages to save performance benchmarks results#1469
codeboten merged 1 commit intoopen-telemetry:masterfrom
NathanielRN:benchmarks-on-gh-pages

Conversation

@NathanielRN
Copy link
Copy Markdown
Contributor

Description

As a follow up to #1443 we found out that the GitHub action will fetch the branch even if it on that branch which produces an error.

This PR goes back to using gh-pages and adds a link in the ReadTheDocs directory to point to the benchmarks graphs which will live on the gh-pages branch.

This is also a good idea to reduce the noise on the master branch.

This will fix the current Core Repo Tests.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

See my example repo which uses gh-pages correctly to publish to https://nathanielrn.github.io/github-benchmarks-example/benchmarks/index.html

Does This PR Require a Contrib Repo Change?

  • No.

Checklist:

  • Followed the style guidelines of this project
  • Documentation has been updated

@NathanielRN NathanielRN requested review from a team, codeboten and ocelotl and removed request for a team December 11, 2020 21:14
Copy link
Copy Markdown
Contributor

@codeboten codeboten left a comment

Choose a reason for hiding this comment

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

👍

@codeboten codeboten added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Dec 11, 2020
@NathanielRN NathanielRN force-pushed the benchmarks-on-gh-pages branch from 01ca210 to 975a4d1 Compare December 11, 2020 22:00
@codeboten codeboten merged commit 2f1f13f into open-telemetry:master Dec 13, 2020
@NathanielRN NathanielRN mentioned this pull request Dec 13, 2020
3 tasks
@NathanielRN NathanielRN deleted the benchmarks-on-gh-pages branch December 14, 2020 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip Changelog PRs that do not require a CHANGELOG.md entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants