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

Add plugin resource handling in tctl #42059

Merged
merged 3 commits into from
May 31, 2024
Merged

Conversation

smallinsky
Copy link
Contributor

@smallinsky smallinsky commented May 27, 2024

What

Add support for plugin resource in tctl.

Note: Plugin resource uses oneof with gogo proto that produce incompatible object using jsonpb and native json encoder.
To support this a customer pluginResourceWrapper was introduced.

changelog: Add support plugin resource in tctl tool

@github-actions github-actions bot added size/md tctl tctl - Teleport admin tool labels May 27, 2024
@github-actions github-actions bot requested review from avatus and capnspacehook May 27, 2024 16:47
@smallinsky smallinsky force-pushed the smallinsky/plugin_create_update branch from 5c50143 to 6739756 Compare May 27, 2024 17:11
Copy link

The PR changelog entry failed validation: Changelog entry not found in the PR body. Please add a "no-changelog" label to the PR, or changelog lines starting with changelog: followed by the changelog entries for the PR.

@smallinsky smallinsky force-pushed the smallinsky/plugin_create_update branch 2 times, most recently from 38f6110 to 29fdc9a Compare May 27, 2024 17:24
@smallinsky smallinsky requested review from r0mant and tcsc May 27, 2024 18:09
tool/tctl/common/collection.go Outdated Show resolved Hide resolved
tool/tctl/common/collection.go Outdated Show resolved Hide resolved
tool/tctl/common/collection.go Outdated Show resolved Hide resolved
tool/tctl/common/resource_command.go Outdated Show resolved Hide resolved
@smallinsky smallinsky force-pushed the smallinsky/plugin_create_update branch from ff9602b to b54b653 Compare May 28, 2024 15:42
@smallinsky smallinsky requested a review from tigrato May 28, 2024 15:42
@smallinsky smallinsky added this pull request to the merge queue May 31, 2024
Merged via the queue into master with commit 87e2075 May 31, 2024
40 checks passed
@smallinsky smallinsky deleted the smallinsky/plugin_create_update branch May 31, 2024 11:01
@public-teleport-github-review-bot

@smallinsky See the table below for backport results.

Branch Result
branch/v15 Failed
branch/v16 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants