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

[liqoctl] generate peering-user #2911

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

claudiolor
Copy link
Contributor

Description

This PR introduces a set of commands allowing to generate a user with the minimum permissions to create a peering from the cluster with the specified cluster ID.

To create an user with the minimum permissions to peer with the current cluster from a cluster with ID $CLUSTER_ID:

liqoctl generate peering-user --consumer-cluster-id $CLUSTER_ID

to remove all the permissions to the user:

liqoctl delete peering-user --consumer-cluster-id $CLUSTER_ID

@adamjensenbot
Copy link
Collaborator

Hi @claudiolor. Thanks for your PR!

I am @adamjensenbot.
You can interact with me issuing a slash command in the first line of a comment.
Currently, I understand the following commands:

  • /rebase: Rebase this PR onto the master branch (You can add the option test=true to launch the tests
    when the rebase operation is completed)
  • /merge: Merge this PR into the master branch
  • /build Build Liqo components
  • /test Launch the E2E and Unit tests
  • /hold, /unhold Add/remove the hold label to prevent merging with /merge

Make sure this PR appears in the liqo changelog, adding one of the following labels:

  • feat: 🚀 New Feature
  • fix: 🐛 Bug Fix
  • refactor: 🧹 Code Refactoring
  • docs: 📝 Documentation
  • style: 💄 Code Style
  • perf: 🐎 Performance Improvement
  • test: ✅ Tests
  • chore: 🚚 Dependencies Management
  • build: 📦 Builds Management
  • ci: 👷 CI/CD
  • revert: ⏪ Reverts Previous Changes

@github-actions github-actions bot added feat Adds a new feature to the codebase fix Fixes a bug in the codebase. style Changes related to code formatting, whitespace, or linting, without affecting functionality labels Jan 29, 2025
@claudiolor claudiolor force-pushed the clo/liqoctl-new-user branch from f19a67a to 3ea3943 Compare January 30, 2025 09:47
@github-actions github-actions bot added docs Updates or adds documentation test Adds or updates tests for the codebase labels Jan 30, 2025
@claudiolor claudiolor force-pushed the clo/liqoctl-new-user branch 8 times, most recently from 0f0e212 to f203cff Compare January 31, 2025 10:09
@claudiolor claudiolor force-pushed the clo/liqoctl-new-user branch 2 times, most recently from 6bb3540 to 063dd70 Compare February 4, 2025 11:18
@claudiolor
Copy link
Contributor Author

/test

This patch introduces a set of commands allowing to generate a user with
the minimum permissions to create a peering from the cluster with the
specified cluster ID
@claudiolor claudiolor force-pushed the clo/liqoctl-new-user branch from b3928aa to 064a423 Compare February 5, 2025 16:38
@claudiolor claudiolor marked this pull request as ready for review February 5, 2025 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Updates or adds documentation feat Adds a new feature to the codebase fix Fixes a bug in the codebase. size/XXL style Changes related to code formatting, whitespace, or linting, without affecting functionality test Adds or updates tests for the codebase
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants