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

Setup plugin #1

Merged
merged 3 commits into from
Jun 7, 2024
Merged

Setup plugin #1

merged 3 commits into from
Jun 7, 2024

Conversation

ADorigi
Copy link
Collaborator

@ADorigi ADorigi commented Jun 7, 2024

Description

This pull request sets up a bare minimum working plugin. The plugin firstly authenticates to your GCP account using gcloud's configuration(more about ADC). It then fetches your instances for the current active project in gcloud. Next, I am trying to configure it to display these instances for the kaytu command kaytu optimize compute-instances. It is still a major work in progress but I would like to understand if my approach is correct.

Steps to test:

  • ensure gcloud is configured, as updated in this repository's Readme.
  • install the gcp plugin in debug mode on kaytu cli
go run main.go plugin install --plugin-debug-mode gcp --unsafe false 
  • run the plugin in debug mode
make debug

Output

Screenshot 2024-06-07 at 12 30 12

What type of PR(Pull Request) is this?

  • 🍕 New Feature
  • 📝 Documentation update
  • ✅ Test
  • 🤖 Build

Added to documentation?

  • 📜 README.md

New Feature

  1. Provide a clear explanation of your new feature.
  2. Include relevant tests for the new feature.
  3. Confirm successful local testing of your changes.
  4. Update the relevant documentation (if applicable).

@acx1729 acx1729 merged commit 419972f into main Jun 7, 2024
@ADorigi ADorigi deleted the setup-plugin branch June 8, 2024 20: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