Skip to content

CDI-171/CDI-172: Plugin Framework Detection and Execution#108

Merged
erikostien-pingidentity merged 9 commits intomainfrom
CDI-421
Jun 11, 2025
Merged

CDI-171/CDI-172: Plugin Framework Detection and Execution#108
erikostien-pingidentity merged 9 commits intomainfrom
CDI-421

Conversation

@erikostien-pingidentity
Copy link
Contributor

  • Add 'protoc' code generation to Makefile, and update some targets for better messaging.
  • Update many test cases to initialize koanf when using options.
  • Add 'plugin' command for managing PingCLI plugins via Hashicorp's go-plugin module.
    This command includes three subcommands 'add', 'list', and 'remove'.
  • Add test cases for new 'plugin' subcommands.
  • Remove init() in root.go command in favor of initialization in the NewRootCommand function. This allows the root cobra command instance to initialize plugins if defined.
  • Add plugin commands to root command on initialization.
  • Update to go 1.24.3 and update various dependencies.
  • Update tests to use PingFederate Terraform Provider 1.5.0.
  • Update tests to use PingOne Terraform Provider 1.8.0.
  • Add Remove() to StringSlice custom type.
  • Add PluginError() output type for plugin related system errors.
  • Host, connect, and dispense plugins in 'internal/plugins/plugins.go'.
  • Use NewKoanfConfig() to initialize main Koanf to avoid odd errors in test cases when run sequentially.
  • Protobuf definitions and generated code located in 'internal/proto'
  • The 'shared' directory contains all code that a golang-based plugin can use to communicate to the PingCLI host process

- Added 'plugin' command and subcommands to manage PingCLI plugins.
- Generated GRPC protobuf code in Makefile for PingCLI and Go-based
Plugins
- Added plugins to root command on start to allow plugins to exist
in usage output
- Added bidirectinoal grpc communication for plugin logging
- Updated Golang to 1.24.3 and updated dependencies
@erikostien-pingidentity erikostien-pingidentity requested a review from a team as a code owner June 3, 2025 00:51
@erikostien-pingidentity erikostien-pingidentity merged commit f85d1eb into main Jun 11, 2025
10 checks passed
@erikostien-pingidentity erikostien-pingidentity deleted the CDI-421 branch June 11, 2025 15:09
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

Comments