Skip to content

Commit

Permalink
fix(oci): add missing method WithBundleParserOpts to oci_downloader_u…
Browse files Browse the repository at this point in the history
…navailable

Signed-off-by: slonka <slonka@users.noreply.github.com>
  • Loading branch information
slonka authored and ashutosh-narkar committed Feb 9, 2024
1 parent 75f3507 commit c8f7244
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 2 deletions.
9 changes: 7 additions & 2 deletions .github/workflows/pull-request.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ jobs:
capabilities.json
go-build:
name: Go Build (${{ matrix.os }}${{ matrix.arch && format(' {0}', matrix.arch) || '' }})
name: Go Build (${{ matrix.os }}${{ matrix.arch && format(' {0}', matrix.arch) || '' }}${{ matrix.go_tags }})
runs-on: ${{ matrix.run }}
needs: generate
strategy:
Expand All @@ -45,6 +45,11 @@ jobs:
run: ubuntu-22.04
targets: ci-go-ci-build-linux-static
arch: arm64
- os: linux
run: ubuntu-22.04
targets: ci-go-ci-build-linux-static
go_tags: GO_TAGS="-tags=opa_no_oci"
arch: arm64
- os: windows
run: ubuntu-22.04
targets: ci-go-ci-build-windows
Expand All @@ -71,7 +76,7 @@ jobs:
name: generated

- name: Build
run: make ${{ matrix.targets }}
run: make ${{ matrix.go_tags }} ${{ matrix.targets }}
env:
GOARCH: ${{ matrix.arch }}
timeout-minutes: 30
Expand Down
6 changes: 6 additions & 0 deletions download/oci_download_unavailable.go
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,8 @@ package download

import (
"context"

"github.com/open-policy-agent/opa/ast"
"github.com/open-policy-agent/opa/bundle"
"github.com/open-policy-agent/opa/plugins/rest"
)
Expand Down Expand Up @@ -51,3 +53,7 @@ func (d *OCIDownloader) Start(context.Context) {
func (d *OCIDownloader) Stop(context.Context) {
panic("built without OCI support")
}

func (*OCIDownloader) WithBundleParserOpts(ast.ParserOptions) *OCIDownloader {
panic("built without OCI support")
}

0 comments on commit c8f7244

Please sign in to comment.