Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate CI to github Actions #403

Merged
merged 2 commits into from
May 30, 2024

Conversation

mantissahz
Copy link
Contributor

@mantissahz mantissahz commented May 24, 2024

e2e-test is not included because there are many errors in my local testing.

We will have the step make e2e-test after the step make ci.

cc @innobead, @bk201, @derekbit, @FrankYang0529

Copy link

@FrankYang0529 FrankYang0529 left a comment

Choose a reason for hiding this comment

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

Overall LGTM. For pushing image, we can merge first and fix it if it can't work.

.github/workflows/build.yml Show resolved Hide resolved
@mantissahz mantissahz force-pushed the migrate-to-gihub-actions branch from 39706c2 to d6ccb91 Compare May 27, 2024 01:59
@derekbit
Copy link
Member

I will merge it after releasing v0.0.27.

@derekbit derekbit added this to the 0.0.28 milestone May 27, 2024
@derekbit
Copy link
Member

@mantissahz
Can you build images for arm and amd64 platforms?

@mantissahz
Copy link
Contributor Author

@derekbit,

I can build arm64 and amd64 images as https://hub.docker.com/repository/docker/jamesluhz/local-path-provisioner/tags.

Do we also need the arm image (it is built in Drone CI)?

@derekbit
Copy link
Member

@derekbit,

I can build arm64 and amd64 images as https://hub.docker.com/repository/docker/jamesluhz/local-path-provisioner/tags.

Do we also need the arm image (it is built in Drone CI)?

Yes, need arm as well.

@mantissahz mantissahz force-pushed the migrate-to-gihub-actions branch from d6ccb91 to 700984a Compare May 29, 2024 05:39
@mantissahz
Copy link
Contributor Author

https://hub.docker.com/repository/docker/jamesluhz/local-path-provisioner/tags
Images built with arch arm64, arm, amd64

@mantissahz mantissahz force-pushed the migrate-to-gihub-actions branch from 700984a to 0676e79 Compare May 29, 2024 06:57
@mantissahz mantissahz requested a review from derekbit May 29, 2024 06:58
@mantissahz mantissahz force-pushed the migrate-to-gihub-actions branch from 0676e79 to bf53d83 Compare May 29, 2024 07:00
@mantissahz mantissahz force-pushed the migrate-to-gihub-actions branch from bf53d83 to 0eabd7f Compare May 29, 2024 07:44
@mantissahz mantissahz force-pushed the migrate-to-gihub-actions branch from 0eabd7f to 6bddefc Compare May 30, 2024 01:39
@mantissahz mantissahz requested a review from FrankYang0529 May 30, 2024 01:39
@derekbit derekbit merged commit 8d20807 into rancher:master May 30, 2024
2 checks passed
coolguy1771 referenced this pull request in coolguy1771/home-ops Jun 19, 2024
…8 ) (#4915)

This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
|
[local-path-provisioner](https://togithub.com/rancher/local-path-provisioner)
| patch | `v0.0.27` -> `v0.0.28` |

---

### Release Notes

<details>
<summary>rancher/local-path-provisioner
(local-path-provisioner)</summary>

###
[`v0.0.28`](https://togithub.com/rancher/local-path-provisioner/releases/tag/v0.0.28):
Local Path Provisioner v0.0.28

[Compare
Source](https://togithub.com/rancher/local-path-provisioner/compare/v0.0.27...v0.0.28)

#### What's Changed

- Migrate CI to github Actions by
[@&#8203;mantissahz](https://togithub.com/mantissahz) in
[https://github.com/rancher/local-path-provisioner/pull/403](https://togithub.com/rancher/local-path-provisioner/pull/403)
- fix(ci): allow to read docker hub secret by
[@&#8203;mantissahz](https://togithub.com/mantissahz) in
[https://github.com/rancher/local-path-provisioner/pull/412](https://togithub.com/rancher/local-path-provisioner/pull/412)
- Revert "Give the helper pod more range of MCS categories" by
[@&#8203;derekbit](https://togithub.com/derekbit) in
[https://github.com/rancher/local-path-provisioner/pull/421](https://togithub.com/rancher/local-path-provisioner/pull/421)
- Temporarily disable TestPodWithMultipleStorageClasses by
[@&#8203;derekbit](https://togithub.com/derekbit) in
[https://github.com/rancher/local-path-provisioner/pull/423](https://togithub.com/rancher/local-path-provisioner/pull/423)
- Move helperPod namespace into metadata by
[@&#8203;justusbunsi](https://togithub.com/justusbunsi) in
[https://github.com/rancher/local-path-provisioner/pull/425](https://togithub.com/rancher/local-path-provisioner/pull/425)

#### New Contributors

- [@&#8203;justusbunsi](https://togithub.com/justusbunsi) made their
first contribution in
[https://github.com/rancher/local-path-provisioner/pull/365](https://togithub.com/rancher/local-path-provisioner/pull/365)
- [@&#8203;mantissahz](https://togithub.com/mantissahz) made their first
contribution in
[https://github.com/rancher/local-path-provisioner/pull/403](https://togithub.com/rancher/local-path-provisioner/pull/403)

**Full Changelog**:
rancher/local-path-provisioner@v0.0.27...v0.0.28

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://togithub.com/renovatebot/renovate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy40MTMuMSIsInVwZGF0ZWRJblZlciI6IjM3LjQxMy4xIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJyZW5vdmF0ZS9naXRodWItcmVsZWFzZSIsInR5cGUvcGF0Y2giXX0=-->

Co-authored-by: lumiere-bot[bot] <98047013+lumiere-bot[bot]@users.noreply.github.com>
lumiere-bot bot referenced this pull request in coolguy1771/home-ops Jun 19, 2024
…8 ) (#4914)

This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
|
[local-path-provisioner](https://togithub.com/rancher/local-path-provisioner)
| patch | `v0.0.27` -> `v0.0.28` |

---

### Release Notes

<details>
<summary>rancher/local-path-provisioner
(local-path-provisioner)</summary>

###
[`v0.0.28`](https://togithub.com/rancher/local-path-provisioner/releases/tag/v0.0.28):
Local Path Provisioner v0.0.28

[Compare
Source](https://togithub.com/rancher/local-path-provisioner/compare/v0.0.27...v0.0.28)

#### What's Changed

- Migrate CI to github Actions by
[@&#8203;mantissahz](https://togithub.com/mantissahz) in
[https://github.com/rancher/local-path-provisioner/pull/403](https://togithub.com/rancher/local-path-provisioner/pull/403)
- fix(ci): allow to read docker hub secret by
[@&#8203;mantissahz](https://togithub.com/mantissahz) in
[https://github.com/rancher/local-path-provisioner/pull/412](https://togithub.com/rancher/local-path-provisioner/pull/412)
- Revert "Give the helper pod more range of MCS categories" by
[@&#8203;derekbit](https://togithub.com/derekbit) in
[https://github.com/rancher/local-path-provisioner/pull/421](https://togithub.com/rancher/local-path-provisioner/pull/421)
- Temporarily disable TestPodWithMultipleStorageClasses by
[@&#8203;derekbit](https://togithub.com/derekbit) in
[https://github.com/rancher/local-path-provisioner/pull/423](https://togithub.com/rancher/local-path-provisioner/pull/423)
- Move helperPod namespace into metadata by
[@&#8203;justusbunsi](https://togithub.com/justusbunsi) in
[https://github.com/rancher/local-path-provisioner/pull/425](https://togithub.com/rancher/local-path-provisioner/pull/425)

#### New Contributors

- [@&#8203;justusbunsi](https://togithub.com/justusbunsi) made their
first contribution in
[https://github.com/rancher/local-path-provisioner/pull/365](https://togithub.com/rancher/local-path-provisioner/pull/365)
- [@&#8203;mantissahz](https://togithub.com/mantissahz) made their first
contribution in
[https://github.com/rancher/local-path-provisioner/pull/403](https://togithub.com/rancher/local-path-provisioner/pull/403)

**Full Changelog**:
rancher/local-path-provisioner@v0.0.27...v0.0.28

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://togithub.com/renovatebot/renovate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy40MTMuMSIsInVwZGF0ZWRJblZlciI6IjM3LjQxMy4xIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJyZW5vdmF0ZS9naXRodWItcmVsZWFzZSIsInR5cGUvcGF0Y2giXX0=-->

Co-authored-by: lumiere-bot[bot] <98047013+lumiere-bot[bot]@users.noreply.github.com>
sp3nx0r referenced this pull request in sp3nx0r/homelab Jun 22, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
|
[local-path-provisioner](https://togithub.com/rancher/local-path-provisioner)
| patch | `v0.0.26` -> `v0.0.28` |

---

### Release Notes

<details>
<summary>rancher/local-path-provisioner
(local-path-provisioner)</summary>

###
[`v0.0.28`](https://togithub.com/rancher/local-path-provisioner/releases/tag/v0.0.28):
Local Path Provisioner v0.0.28

[Compare
Source](https://togithub.com/rancher/local-path-provisioner/compare/v0.0.27...v0.0.28-rc1)

#### What's Changed

- Migrate CI to github Actions by
[@&#8203;mantissahz](https://togithub.com/mantissahz) in
[https://github.com/rancher/local-path-provisioner/pull/403](https://togithub.com/rancher/local-path-provisioner/pull/403)
- fix(ci): allow to read docker hub secret by
[@&#8203;mantissahz](https://togithub.com/mantissahz) in
[https://github.com/rancher/local-path-provisioner/pull/412](https://togithub.com/rancher/local-path-provisioner/pull/412)
- Revert "Give the helper pod more range of MCS categories" by
[@&#8203;derekbit](https://togithub.com/derekbit) in
[https://github.com/rancher/local-path-provisioner/pull/421](https://togithub.com/rancher/local-path-provisioner/pull/421)
- Temporarily disable TestPodWithMultipleStorageClasses by
[@&#8203;derekbit](https://togithub.com/derekbit) in
[https://github.com/rancher/local-path-provisioner/pull/423](https://togithub.com/rancher/local-path-provisioner/pull/423)
- Move helperPod namespace into metadata by
[@&#8203;justusbunsi](https://togithub.com/justusbunsi) in
[https://github.com/rancher/local-path-provisioner/pull/425](https://togithub.com/rancher/local-path-provisioner/pull/425)

#### New Contributors

- [@&#8203;justusbunsi](https://togithub.com/justusbunsi) made their
first contribution in
[https://github.com/rancher/local-path-provisioner/pull/365](https://togithub.com/rancher/local-path-provisioner/pull/365)
- [@&#8203;mantissahz](https://togithub.com/mantissahz) made their first
contribution in
[https://github.com/rancher/local-path-provisioner/pull/403](https://togithub.com/rancher/local-path-provisioner/pull/403)

**Full Changelog**:
rancher/local-path-provisioner@v0.0.27...v0.0.28

###
[`v0.0.27`](https://togithub.com/rancher/local-path-provisioner/releases/tag/v0.0.27):
Local Path Provisioner v0.0.27

[Compare
Source](https://togithub.com/rancher/local-path-provisioner/compare/v0.0.26...v0.0.27)

#### What's Changed

- Repair code example in storageClass description. by
[@&#8203;c4lliope](https://togithub.com/c4lliope) in
[https://github.com/rancher/local-path-provisioner/pull/368](https://togithub.com/rancher/local-path-provisioner/pull/368)
- Update README.md to 0.26 by
[@&#8203;e-minguez](https://togithub.com/e-minguez) in
[https://github.com/rancher/local-path-provisioner/pull/373](https://togithub.com/rancher/local-path-provisioner/pull/373)
- Update README.md,rm /manager by
[@&#8203;terryzwt](https://togithub.com/terryzwt) in
[https://github.com/rancher/local-path-provisioner/pull/379](https://togithub.com/rancher/local-path-provisioner/pull/379)
- Fix duplicate labels by
[@&#8203;runningman84](https://togithub.com/runningman84) in
[https://github.com/rancher/local-path-provisioner/pull/393](https://togithub.com/rancher/local-path-provisioner/pull/393)
- drone: remove s390x support by
[@&#8203;derekbit](https://togithub.com/derekbit) in
[https://github.com/rancher/local-path-provisioner/pull/391](https://togithub.com/rancher/local-path-provisioner/pull/391)
- Feature/multiple storage classes by
[@&#8203;meln5674](https://togithub.com/meln5674) in
[https://github.com/rancher/local-path-provisioner/pull/361](https://togithub.com/rancher/local-path-provisioner/pull/361)
- Remove duplicate labels and add ability to set helperpod resource
requests/limits by [@&#8203;visokoo](https://togithub.com/visokoo) in
[https://github.com/rancher/local-path-provisioner/pull/394](https://togithub.com/rancher/local-path-provisioner/pull/394)
- Automatic reloading of the helper pod manifest by the provisioner by
[@&#8203;js185692](https://togithub.com/js185692) in
[https://github.com/rancher/local-path-provisioner/pull/399](https://togithub.com/rancher/local-path-provisioner/pull/399)
- Add support for custom path patterns by
[@&#8203;AlbanBedel](https://togithub.com/AlbanBedel) in
[https://github.com/rancher/local-path-provisioner/pull/385](https://togithub.com/rancher/local-path-provisioner/pull/385)
- adding pvc with node name example by
[@&#8203;sebastianohl](https://togithub.com/sebastianohl) in
[https://github.com/rancher/local-path-provisioner/pull/382](https://togithub.com/rancher/local-path-provisioner/pull/382)
- Add e2e test for custom path patterns by
[@&#8203;derekbit](https://togithub.com/derekbit) in
[https://github.com/rancher/local-path-provisioner/pull/404](https://togithub.com/rancher/local-path-provisioner/pull/404)
- Give the helper pod more range of MCS categories by
[@&#8203;galal-hussein](https://togithub.com/galal-hussein) in
[https://github.com/rancher/local-path-provisioner/pull/402](https://togithub.com/rancher/local-path-provisioner/pull/402)
- Fix: Chart.yaml file is missing on helm install by
[@&#8203;jamshidi799](https://togithub.com/jamshidi799) in
[https://github.com/rancher/local-path-provisioner/pull/388](https://togithub.com/rancher/local-path-provisioner/pull/388)
- drone: disable e2e test by
[@&#8203;derekbit](https://togithub.com/derekbit) in
[https://github.com/rancher/local-path-provisioner/pull/405](https://togithub.com/rancher/local-path-provisioner/pull/405)
- Allow customizing helper pod by
[@&#8203;justusbunsi](https://togithub.com/justusbunsi) in
[https://github.com/rancher/local-path-provisioner/pull/365](https://togithub.com/rancher/local-path-provisioner/pull/365)
- test: use reclaimPolicy Delete instead by
[@&#8203;derekbit](https://togithub.com/derekbit) in
[https://github.com/rancher/local-path-provisioner/pull/406](https://togithub.com/rancher/local-path-provisioner/pull/406)
- chart: fix pathPattern by
[@&#8203;derekbit](https://togithub.com/derekbit) in
[https://github.com/rancher/local-path-provisioner/pull/409](https://togithub.com/rancher/local-path-provisioner/pull/409)

#### New Contributors

- [@&#8203;c4lliope](https://togithub.com/c4lliope) made their first
contribution in
[https://github.com/rancher/local-path-provisioner/pull/368](https://togithub.com/rancher/local-path-provisioner/pull/368)
- [@&#8203;e-minguez](https://togithub.com/e-minguez) made their first
contribution in
[https://github.com/rancher/local-path-provisioner/pull/373](https://togithub.com/rancher/local-path-provisioner/pull/373)
- [@&#8203;terryzwt](https://togithub.com/terryzwt) made their first
contribution in
[https://github.com/rancher/local-path-provisioner/pull/379](https://togithub.com/rancher/local-path-provisioner/pull/379)
- [@&#8203;runningman84](https://togithub.com/runningman84) made their
first contribution in
[https://github.com/rancher/local-path-provisioner/pull/393](https://togithub.com/rancher/local-path-provisioner/pull/393)
- [@&#8203;visokoo](https://togithub.com/visokoo) made their first
contribution in
[https://github.com/rancher/local-path-provisioner/pull/394](https://togithub.com/rancher/local-path-provisioner/pull/394)
- [@&#8203;AlbanBedel](https://togithub.com/AlbanBedel) made their first
contribution in
[https://github.com/rancher/local-path-provisioner/pull/385](https://togithub.com/rancher/local-path-provisioner/pull/385)
- [@&#8203;sebastianohl](https://togithub.com/sebastianohl) made their
first contribution in
[https://github.com/rancher/local-path-provisioner/pull/382](https://togithub.com/rancher/local-path-provisioner/pull/382)
- [@&#8203;galal-hussein](https://togithub.com/galal-hussein) made their
first contribution in
[https://github.com/rancher/local-path-provisioner/pull/402](https://togithub.com/rancher/local-path-provisioner/pull/402)
- [@&#8203;jamshidi799](https://togithub.com/jamshidi799) made their
first contribution in
[https://github.com/rancher/local-path-provisioner/pull/388](https://togithub.com/rancher/local-path-provisioner/pull/388)
- [@&#8203;justusbunsi](https://togithub.com/justusbunsi) made their
first contribution in
[https://github.com/rancher/local-path-provisioner/pull/365](https://togithub.com/rancher/local-path-provisioner/pull/365)

**Full Changelog**:
rancher/local-path-provisioner@v0.0.26...v0.0.27

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "on saturday" (UTC), Automerge - At
any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/sp3nx0r/homelab).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zNzcuOCIsInVwZGF0ZWRJblZlciI6IjM3LjQxMy4yIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJyZW5vdmF0ZS9naXRodWItcmVsZWFzZSIsInR5cGUvcGF0Y2giXX0=-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants