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

feat(service): added support for labels on serviceSpec #202

Merged
merged 1 commit into from
Jun 18, 2024

Conversation

thecodeassassin
Copy link
Contributor

Tailscale services require labels to assign a ProxyClass.

This PR adds support for labels on serviceSpec to accomplish this goal.

Copy link
Contributor

@Abhra303 Abhra303 left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! Mostly looks good. See some comments.

api/v1alpha1/dragonfly_types.go Outdated Show resolved Hide resolved
manifests/crd.yaml Outdated Show resolved Hide resolved
@thecodeassassin
Copy link
Contributor Author

@Abhra303 Should be good now, changes in manifests reverted

manifests/crd.yaml Outdated Show resolved Hide resolved
@Abhra303
Copy link
Contributor

Could you please sign the commits so that I can merge?

@thecodeassassin
Copy link
Contributor Author

Could you please sign the commits so that I can merge?

I am sorry, i don't know what you mean.

@Abhra303
Copy link
Contributor

To merge your PRs, you need to sign your commits meaning you agree with our licensing terms and conditions. See https://github.com/dragonflydb/dragonfly/blob/main/CONTRIBUTING.md#signoff-commits.

You can rebase all the commits into a single commit and sign it with -s flag or edit each commit and sign.
Example to signoff last four commits:

git rebase --signoff HEAD~4

https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits

@thecodeassassin
Copy link
Contributor Author

I think that should be done ? Can you please check? never signed commits before.

@Abhra303
Copy link
Contributor

Abhra303 commented Jun 18, 2024

Looks like you didn't remove the old commits that are not signed, can you please remove those?

Also you should use your full name to sign ( I see only Stephen here). You can set your full name by git config user.name "Your FullName". Then sign the commits again. Additionally you can also set PGP key to sign commits (See https://codex.so/gpg-verification-github). Signing with PGP keys verify that its really you and not an imposter signing on behalf of you. Its preferred to have the PGP key.

never signed commits before.

No issuse, take your time. Its really important to sign your commits (most of the open source repos require signed commits). Once you set up this, its very easy to sign commits (just add a -S flag while committing changes). Don't hesitate to ask me if you are stuck anywhere (e.g. PGP setup/ signing, removing the old unsigned commits etc.).

@thecodeassassin
Copy link
Contributor Author

@

Looks like you didn't remove the old commits that are not signed, can you please remove those?

Also you should use your full name to sign ( I see only Stephen here). You can set your full name by git config user.name "Your FullName". Then sign the commits again. Additionally you can also set PGP key to sign commits (See https://codex.so/gpg-verification-github). Signing with PGP keys verify that its really you and not an imposter signing on behalf of you. Its preferred to have the PGP key.

never signed commits before.

No issuse, take your time. Its really important to sign your commits (most of the open source repos require signed commits). Once you set up this, its very easy to sign commits (just add a -S flag while committing changes). Don't hesitate to ask me if you are stuck anywhere (e.g. PGP setup/ signing, removing the old unsigned commits etc.).

Allright, should be good now

@Abhra303
Copy link
Contributor

Thanks! It will work!

@Abhra303 Abhra303 merged commit f399e7e into dragonflydb:main Jun 18, 2024
2 checks passed
szinn referenced this pull request in szinn/k8s-homelab Jun 26, 2024
…3895)

This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| docker.dragonflydb.io/dragonflydb/operator | patch | `v1.1.3` ->
`v1.1.4` |
|
[dragonflydb/dragonfly-operator](https://togithub.com/dragonflydb/dragonfly-operator)
| patch | `v1.1.3` -> `v1.1.4` |

---

### Release Notes

<details>
<summary>dragonflydb/dragonfly-operator
(dragonflydb/dragonfly-operator)</summary>

###
[`v1.1.4`](https://togithub.com/dragonflydb/dragonfly-operator/releases/tag/v1.1.4)

[Compare
Source](https://togithub.com/dragonflydb/dragonfly-operator/compare/v1.1.3...v1.1.4)

Release v1.1.4

Docker image: `docker.dragonflydb.io/dragonflydb/operator:v1.1.4`

This release adds support for custom service labels and fix a bug for
single-pod dragonfly rolling updates.

##### What's Changed

- fix(rolling-update): avoid replica takeover for single pod instance by
[@&#8203;Abhra303](https://togithub.com/Abhra303) in
[https://github.com/dragonflydb/dragonfly-operator/pull/203](https://togithub.com/dragonflydb/dragonfly-operator/pull/203)
- feat(service): added support for labels on serviceSpec by
[@&#8203;thecodeassassin](https://togithub.com/thecodeassassin) in
[https://github.com/dragonflydb/dragonfly-operator/pull/202](https://togithub.com/dragonflydb/dragonfly-operator/pull/202)
- feat(version): release 1.1.4 of Operator by
[@&#8203;Abhra303](https://togithub.com/Abhra303) in
[https://github.com/dragonflydb/dragonfly-operator/pull/206](https://togithub.com/dragonflydb/dragonfly-operator/pull/206)

##### New Contributors

- [@&#8203;thecodeassassin](https://togithub.com/thecodeassassin) made
their first contribution in
[https://github.com/dragonflydb/dragonfly-operator/pull/202](https://togithub.com/dragonflydb/dragonfly-operator/pull/202)

**Full Changelog**:
dragonflydb/dragonfly-operator@v1.1.3...v1.1.4

</details>

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

Co-authored-by: repo-jeeves[bot] <106431701+repo-jeeves[bot]@users.noreply.github.com>
kireque referenced this pull request in kireque/home-ops Jun 26, 2024
…patch) (#712)

This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| docker.dragonflydb.io/dragonflydb/operator | patch | `v1.1.3` ->
`v1.1.4` |
|
[dragonflydb/dragonfly-operator](https://togithub.com/dragonflydb/dragonfly-operator)
| patch | `v1.1.3` -> `v1.1.4` |

---

### Release Notes

<details>
<summary>dragonflydb/dragonfly-operator
(dragonflydb/dragonfly-operator)</summary>

###
[`v1.1.4`](https://togithub.com/dragonflydb/dragonfly-operator/releases/tag/v1.1.4)

[Compare
Source](https://togithub.com/dragonflydb/dragonfly-operator/compare/v1.1.3...v1.1.4)

Release v1.1.4

Docker image: `docker.dragonflydb.io/dragonflydb/operator:v1.1.4`

This release adds support for custom service labels and fix a bug for
single-pod dragonfly rolling updates.

##### What's Changed

- fix(rolling-update): avoid replica takeover for single pod instance by
[@&#8203;Abhra303](https://togithub.com/Abhra303) in
[https://github.com/dragonflydb/dragonfly-operator/pull/203](https://togithub.com/dragonflydb/dragonfly-operator/pull/203)
- feat(service): added support for labels on serviceSpec by
[@&#8203;thecodeassassin](https://togithub.com/thecodeassassin) in
[https://github.com/dragonflydb/dragonfly-operator/pull/202](https://togithub.com/dragonflydb/dragonfly-operator/pull/202)
- feat(version): release 1.1.4 of Operator by
[@&#8203;Abhra303](https://togithub.com/Abhra303) in
[https://github.com/dragonflydb/dragonfly-operator/pull/206](https://togithub.com/dragonflydb/dragonfly-operator/pull/206)

##### New Contributors

- [@&#8203;thecodeassassin](https://togithub.com/thecodeassassin) made
their first contribution in
[https://github.com/dragonflydb/dragonfly-operator/pull/202](https://togithub.com/dragonflydb/dragonfly-operator/pull/202)

**Full Changelog**:
dragonflydb/dragonfly-operator@v1.1.3...v1.1.4

</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 these
updates 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:eyJjcmVhdGVkSW5WZXIiOiIzNy40MTguMCIsInVwZGF0ZWRJblZlciI6IjM3LjQxOC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJyZW5vdmF0ZS9jb250YWluZXIiLCJyZW5vdmF0ZS9naXRodWItcmVsZWFzZSIsInR5cGUvcGF0Y2giXX0=-->

Co-authored-by: kireque-bot[bot] <143391978+kireque-bot[bot]@users.noreply.github.com>
lumiere-bot bot referenced this pull request in coolguy1771/home-ops Jun 27, 2024
…4 ) (patch) (#4945)

This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
|
[dragonflydb/dragonfly-operator](https://togithub.com/dragonflydb/dragonfly-operator)
| patch | `v1.1.3` -> `v1.1.4` |
| ghcr.io/dragonflydb/operator | patch | `v1.1.3` -> `v1.1.4` |

---

### Release Notes

<details>
<summary>dragonflydb/dragonfly-operator
(dragonflydb/dragonfly-operator)</summary>

###
[`v1.1.4`](https://togithub.com/dragonflydb/dragonfly-operator/releases/tag/v1.1.4)

[Compare
Source](https://togithub.com/dragonflydb/dragonfly-operator/compare/v1.1.3...v1.1.4)

Release v1.1.4

Docker image: `docker.dragonflydb.io/dragonflydb/operator:v1.1.4`

This release adds support for custom service labels and fix a bug for
single-pod dragonfly rolling updates.

##### What's Changed

- fix(rolling-update): avoid replica takeover for single pod instance by
[@&#8203;Abhra303](https://togithub.com/Abhra303) in
[https://github.com/dragonflydb/dragonfly-operator/pull/203](https://togithub.com/dragonflydb/dragonfly-operator/pull/203)
- feat(service): added support for labels on serviceSpec by
[@&#8203;thecodeassassin](https://togithub.com/thecodeassassin) in
[https://github.com/dragonflydb/dragonfly-operator/pull/202](https://togithub.com/dragonflydb/dragonfly-operator/pull/202)
- feat(version): release 1.1.4 of Operator by
[@&#8203;Abhra303](https://togithub.com/Abhra303) in
[https://github.com/dragonflydb/dragonfly-operator/pull/206](https://togithub.com/dragonflydb/dragonfly-operator/pull/206)

##### New Contributors

- [@&#8203;thecodeassassin](https://togithub.com/thecodeassassin) made
their first contribution in
[https://github.com/dragonflydb/dragonfly-operator/pull/202](https://togithub.com/dragonflydb/dragonfly-operator/pull/202)

**Full Changelog**:
dragonflydb/dragonfly-operator@v1.1.3...v1.1.4

</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 these
updates 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:eyJjcmVhdGVkSW5WZXIiOiIzNy40MTguMCIsInVwZGF0ZWRJblZlciI6IjM3LjQxOC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJyZW5vdmF0ZS9jb250YWluZXIiLCJyZW5vdmF0ZS9naXRodWItcmVsZWFzZSIsInR5cGUvcGF0Y2giXX0=-->

Co-authored-by: lumiere-bot[bot] <98047013+lumiere-bot[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.

2 participants