Skip to content

Commit

Permalink
chore: remove yq from release-create-release.yaml (#1291) (#1295)
Browse files Browse the repository at this point in the history
Remove yq usage from release-create-release.yaml workflow
Remove yq from Makefile

Co-authored-by: Kamil Kasperski <[email protected]>
  • Loading branch information
werdes72 and Ressetkk authored Feb 11, 2025
1 parent a83240d commit 904ba7a
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 21 deletions.
15 changes: 1 addition & 14 deletions .github/workflows/release-create-release.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -21,23 +21,10 @@ jobs:
uses: actions/checkout@v4
with:
fetch-depth: 0

- name: Install yq
shell: bash
run: |
sudo add-apt-repository ppa:rmescandon/yq
sudo apt update
sudo apt install yq -y
- name: Check security-scanner config
shell: bash
run: |
if [[ $( yq eval ".protecode[0]" sec-scanners-config.yaml ) == "europe-docker.pkg.dev/kyma-project/prod/istio/releases/istio-manager:${{ github.event.inputs.version }}" ]]; then
exit 0
else
echo "Error: istio-manager image tag in sec-scanners-config doesn't match release ${{ github.event.inputs.version }}"
exit 1
fi
cat sec-scanners-config.yaml | grep -q "europe-docker.pkg.dev/kyma-project/prod/istio/releases/istio-manager:${{ github.event.inputs.version }}"
build-image:
uses: kyma-project/test-infra/.github/workflows/image-builder.yml@main
Expand Down
7 changes: 0 additions & 7 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -164,12 +164,10 @@ $(LOCALBIN):
KUSTOMIZE ?= $(LOCALBIN)/kustomize
CONTROLLER_GEN ?= $(LOCALBIN)/controller-gen
ENVTEST ?= $(LOCALBIN)/setup-envtest
YQUERY ?= $(LOCALBIN)/yq

## Tool Versions
KUSTOMIZE_VERSION ?= v5.5.0
CONTROLLER_TOOLS_VERSION ?= v0.16.5
YQ_VERSION ?= v4

KUSTOMIZE_INSTALL_SCRIPT ?= "https://raw.githubusercontent.com/kubernetes-sigs/kustomize/master/hack/install_kustomize.sh"
.PHONY: kustomize
Expand All @@ -182,11 +180,6 @@ controller-gen: $(CONTROLLER_GEN) ## Download controller-gen locally if necessar
$(CONTROLLER_GEN): $(LOCALBIN)
test -s $(LOCALBIN)/controller-gen || GOBIN=$(LOCALBIN) go install sigs.k8s.io/controller-tools/cmd/controller-gen@$(CONTROLLER_TOOLS_VERSION)

.PHONY: yq
yq: $(YQUERY) ## Download yq locally if necessary.
$(YQUERY): $(LOCALBIN)
test -s $(LOCALBIN)/yq || { go get github.com/mikefarah/yq/$(YQ_VERSION) ; GOBIN=$(LOCALBIN) go install github.com/mikefarah/yq/$(YQ_VERSION) ; }

.PHONY: envtest
envtest: $(ENVTEST) ## Download envtest-setup locally if necessary.
$(ENVTEST): $(LOCALBIN)
Expand Down

0 comments on commit 904ba7a

Please sign in to comment.