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

[v17] backport netiq implementation #51541

Merged
merged 7 commits into from
Jan 29, 2025
Merged

Conversation

tigrato
Copy link
Contributor

@tigrato tigrato commented Jan 28, 2025

Backport #50476 to branch/v17
Backport #50477 to branch/v17
Backport #51285 to branch/v17
Backport #51290 to branch/v17
Backport #51502 to branch/v17

tigrato and others added 4 commits January 28, 2025 10:58
This PR introduces the necessary protobuf changes to accomodate NetIQ
plugin.

Part of gravitational/access-graph#634

Signed-off-by: Tiago Silva <[email protected]>
* netiq: add access graph protobufs

This PR introduces the necessary protobuf changes to accommodate Access Graph receiving NetIQ stream of changes.

Part of gravitational/access-graph#634

Signed-off-by: Tiago Silva <[email protected]>

* Update netiq.proto

Co-authored-by: Gabriel Corado <[email protected]>

---------

Signed-off-by: Tiago Silva <[email protected]>
Co-authored-by: Gabriel Corado <[email protected]>
This PR exposes the `GetNetIq` method in `PluginStatus` interface to be
used in `e`.

Signed-off-by: Tiago Silva <[email protected]>
@tigrato tigrato added the no-changelog Indicates that a PR does not require a changelog entry label Jan 28, 2025
@tigrato tigrato marked this pull request as draft January 28, 2025 11:03
* netiq: add plugin install prompt

This PR adds the plugin prompt code to read the required parameters to
create the netiq plugin.

Part of gravitational/access-graph#634

Signed-off-by: Tiago Silva <[email protected]>

* add license

* Update netiq.go

Co-authored-by: Gavin Frazar <[email protected]>

* Update netiq.go

Co-authored-by: Gavin Frazar <[email protected]>

* Update netiq.go

Co-authored-by: Gavin Frazar <[email protected]>

* handle review comments

* Apply suggestions from code review

Co-authored-by: Gavin Frazar <[email protected]>

* Apply suggestions from code review

Co-authored-by: Gavin Frazar <[email protected]>

* consolidate prompt methods

---------

Signed-off-by: Tiago Silva <[email protected]>
Co-authored-by: Gavin Frazar <[email protected]>
@tigrato tigrato marked this pull request as ready for review January 29, 2025 19:32
@github-actions github-actions bot added size/lg tctl tctl - Teleport admin tool labels Jan 29, 2025
@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from strideynet January 29, 2025 19:33
@tigrato tigrato enabled auto-merge January 29, 2025 19:35
@tigrato tigrato added this pull request to the merge queue Jan 29, 2025
Merged via the queue into branch/v17 with commit 6d7421f Jan 29, 2025
42 checks passed
@tigrato tigrato deleted the tigrato/netiq-backport-v17 branch January 29, 2025 20:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport no-changelog Indicates that a PR does not require a changelog entry size/lg size/md tctl tctl - Teleport admin tool
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants