Skip to content

Commit

Permalink
fix: folder does not exist (#527)
Browse files Browse the repository at this point in the history
## Description

<!-- 
Please do not leave this blank 
This PR [adds/removes/fixes/replaces] the [feature/bug/etc]. 
-->

Please include a summary of the changes and the related issue. Please
also include relevant motivation and context. List any dependencies that
are required for this change.


## What type of PR is this? (check all applicable)

- [ ] 🍕 Feature
- [ ] 🐛 Bug Fix
- [ ] 📝 Documentation Update
- [ ] 🎨 Style
- [ ] 🧑‍💻 Code Refactor
- [ ] 🔥 Performance Improvements
- [ ] ✅ Test
- [ ] 🤖 Build
- [ ] 🔁 CI
- [ ] 📦 Chore (Release)
- [ ] ⏩ Revert

## Related Tickets & Documents

<!-- 
Please use this format link issue numbers: Fixes #123

https://docs.github.com/en/free-pro-team@latest/github/managing-your-work-on-github/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword
-->
- Related Issue # (issue)
- Closes # (issue)
- Fixes # (issue)
> Remove if not applicable

## Screenshots

<!-- Visual changes require screenshots -->


## Added tests?

- [ ] 👍 yes
- [ ] 🙅 no, because they aren't needed
- [ ] 🙋 no, because I need help
- [ ] Separate ticket for tests # (issue/pr)

Please describe the tests that you ran to verify your changes. Provide
instructions so we can reproduce. Please also list any relevant details
for your test configuration


## Added to documentation?

- [ ] 📜 README.md
- [ ] 🙅 no documentation needed

## Checklist:

- [ ] My code follows the style guidelines of this project
- [ ] I have performed a self-review of my code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
- [ ] My changes generate no new warnings
- [ ] I have added tests that prove my fix is effective or that my
feature works
- [ ] New and existing unit tests pass locally with my changes
- [ ] Any dependent changes have been merged and published in downstream
modules
  • Loading branch information
Skarlso authored Oct 3, 2024
1 parent 020f60b commit a7d5ad6
Show file tree
Hide file tree
Showing 6 changed files with 43 additions and 16 deletions.
37 changes: 37 additions & 0 deletions .github/workflows/components_push.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,37 @@
name: component CTFs

on:
workflow_dispatch:
workflow_call:

permissions:
contents: read
pull-requests: read

jobs:
build-cli:
name: Build component
runs-on: large_runner
steps:
- name: Checkout
uses: actions/checkout@v4
with:
ref: ${{ github.event.pull_request.head.sha }}
fetch-depth: 0
- name: Cache go-build and mod
uses: actions/cache@v4
with:
path: |
~/.cache/go-build/
~/go/pkg/mod/
key: go-${{ hashFiles('go.sum') }}
restore-keys: |
go-
- name: Setup Go
uses: actions/setup-go@v5
with:
go-version-file: '${{ github.workspace }}/go.mod'
- name: CTF
run: |
cd component/
PATH=$PATH:$(go env GOPATH)/bin make ocm && make ctf && make push
Original file line number Diff line number Diff line change
Expand Up @@ -38,4 +38,4 @@ jobs:
- name: CTF
run: |
cd component/
PATH=$PATH:$(go env GOPATH)/bin make ocm && make ctf
PATH=$PATH:$(go env GOPATH)/bin make ocm && make ctf
2 changes: 1 addition & 1 deletion .github/workflows/release.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ jobs:
pull-requests: 'read'
components:
name: Component CTF Builds
uses: ./.github/workflows/components.yaml
uses: ./.github/workflows/components_push.yaml
permissions:
contents: read
pull-requests: read
Expand Down
13 changes: 0 additions & 13 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -161,19 +161,6 @@ dev-deploy: kustomize ## Deploy controller dev image in the configured Kubernete
undeploy: ## Undeploy controller from the K8s cluster specified in ~/.kube/config. Call with ignore-not-found=true to ignore resource not found errors during deletion.
$(KUSTOMIZE) build config/default | kubectl delete --ignore-not-found=$(ignore-not-found) -f -


##@ Component

GEN ?= $(REPO_ROOT)/gen

$(GEN)/.exists:
@mkdir -p $(GEN)
@touch $@

.PHONY: plain-push
plain-push: ocm $(GEN)/.exists
$(OCM) transfer ctf -f $(GEN)/component/ctf $(OCMREPO)

##@ Documentation

api-docs: gen-crd-api-reference-docs ## Generate API reference documentation
Expand Down
3 changes: 3 additions & 0 deletions docs/release_notes/v0.24.7.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
# Release 0.24.7

- fix: making sure to test on pushes but push on a release
2 changes: 1 addition & 1 deletion pkg/version/release.go
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
package version

// ReleaseVersion is the version number in semver format "vX.Y.Z", prefixed with "v".
var ReleaseVersion = "v0.24.6"
var ReleaseVersion = "v0.24.7"

// ReleaseCandidate is the release candidate ID in format "rc.X", which will be appended to the release version.
var ReleaseCandidate = "rc.1"

0 comments on commit a7d5ad6

Please sign in to comment.