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

Move to frostdb on a private fork #58

Merged
merged 11 commits into from
Jun 28, 2022
Merged

Move to frostdb on a private fork #58

merged 11 commits into from
Jun 28, 2022

Conversation

cyriltovena
Copy link
Collaborator

This will help us speed up the development cycle and test out some fixes for frostDB.

Still need to rebase and fix the new select merge API.

@cyriltovena cyriltovena marked this pull request as ready for review June 27, 2022 14:09
@cyriltovena cyriltovena mentioned this pull request Jun 27, 2022
@cyriltovena cyriltovena requested a review from simonswine June 28, 2022 07:29
Copy link
Collaborator

@simonswine simonswine left a comment

Choose a reason for hiding this comment

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

LGTM

Comment on lines 54 to 57
- name: Configure git for private modules
env:
TOKEN: ${{ secrets.GRAFANABOT_PAT }}
run: git config --global url."https://grafanabot:${TOKEN}@github.com".insteadOf "https://github.com"
Copy link
Collaborator

Choose a reason for hiding this comment

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

I would use the same approach to both pipelines. But you might already have tried that?

Suggested change
- name: Configure git for private modules
env:
TOKEN: ${{ secrets.GRAFANABOT_PAT }}
run: git config --global url."https://grafanabot:${TOKEN}@github.com".insteadOf "https://github.com"
- uses: webfactory/[email protected]
with:
ssh-private-key: ${{ secrets.GO_MOD_SSH_PRIVATE_KEY }}

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

trying

@cyriltovena cyriltovena enabled auto-merge June 28, 2022 07:49
@cyriltovena cyriltovena merged commit 738b8cc into main Jun 28, 2022
@cyriltovena cyriltovena deleted the private-frostdb branch June 28, 2022 07:56
simonswine pushed a commit to simonswine/pyroscope that referenced this pull request Jun 30, 2023
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