Skip to content

Commit

Permalink
fix: fetch the correct repos
Browse files Browse the repository at this point in the history
  • Loading branch information
trevorwhitney committed Feb 8, 2024
1 parent d35c5cc commit ae60528
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 6 deletions.
13 changes: 9 additions & 4 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,11 @@ jobs:
- "shouldRelease"
runs-on: "ubuntu-latest"
steps:
- name: "pull code to release"
- name: "pull release library code"
uses: "actions/checkout@v4"
with:
path: "release"
repository: "${{ inputs.release_repo }}"
path: "lib"
repository: "grafana/loki-release"
- name: "setup node"
uses: "actions/setup-node@v4"
with:
Expand Down Expand Up @@ -42,7 +42,7 @@ jobs:
--target-branch "${{ needs.shouldRelease.outputs.branch }}" \
--token="${{ secrets.GH_TOKEN }}"
shell: "bash"
working-directory: "release"
working-directory: "lib"
- env:
GH_TOKEN: "${{ secrets.GH_TOKEN }}"
id: "upload"
Expand Down Expand Up @@ -98,6 +98,11 @@ jobs:
shouldRelease: "${{ steps.should_release.outputs.shouldRelease }}"
runs-on: "ubuntu-latest"
steps:
- name: "pull code to release"
uses: "actions/checkout@v4"
with:
path: "release"
repository: "${{ inputs.release_repo }}"
- name: "pull release library code"
uses: "actions/checkout@v4"
with:
Expand Down
5 changes: 3 additions & 2 deletions workflows/lib/release.libsonnet
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@ local pullRequestFooter = 'Merging this PR will release the [artifacts](https://

shouldRelease: job.new()
+ job.withSteps([
common.fetchReleaseRepo,
common.fetchReleaseLib,
common.extractBranchName,

Expand All @@ -64,7 +65,7 @@ local pullRequestFooter = 'Merging this PR will release the [artifacts](https://
+ job.withNeeds(['shouldRelease'])
+ job.withIf('${{ fromJSON(needs.shouldRelease.outputs.shouldRelease) }}')
+ job.withSteps([
common.fetchReleaseRepo,
common.fetchReleaseLib,
common.setupNode,
common.googleAuth,
common.setupGoogleCloudSdk,
Expand All @@ -81,7 +82,7 @@ local pullRequestFooter = 'Merging this PR will release the [artifacts](https://
gsutil cp -r gs://loki-build-artifacts/${SHA}/dist .
|||),

releaseStep('create release')
releaseLibStep('create release')
+ step.withId('release')
+ step.withRun(|||
npm install
Expand Down

0 comments on commit ae60528

Please sign in to comment.