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

vdk-plugins: Introduce vdk-jobs-troubleshooting plugin #1428

Merged
merged 1 commit into from
Dec 13, 2022

Conversation

doks5
Copy link
Contributor

@doks5 doks5 commented Dec 12, 2022

This is the initial implementation of the vdk-jobs-troubleshooting plugin as part of VEP-1416. This is only the frame of the plugin, actual troubleshooting logic will be added in subsequent PRs.

Testing Done: Trivial, no logic to test yet.

Signed-off-by: Andon Andonov andonova@vmware.com

@doks5 doks5 force-pushed the person/andonova/vdk-job-troubleshooting-plugin branch from 58c457e to 1c7ce4f Compare December 12, 2022 15:27
@murphp15
Copy link
Collaborator

Why are you adding the Skelton and the logic separately?

@doks5
Copy link
Contributor Author

doks5 commented Dec 12, 2022

Why are you adding the Skelton and the logic separately?

To avoid overwhelming the PRs with logic that is mostly the same for all plugins.

Copy link
Collaborator

@antoniivanov antoniivanov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please address my CI comment. But looks good to me.

@antoniivanov
Copy link
Collaborator

Did you use cookiecutter https://github.com/tozka/cookiecutter-vdk-plugin ?

This is the initial implementation of the vdk-jobs-troubleshooting plugin as
part of VEP-1416. This is only the frame of the plugin, actual troubleshooting logic
will be added in subsequent PRs.

Testing Done: Trivial, no logic to test yet.

Signed-off-by: Andon Andonov <andonova@vmware.com>
@doks5 doks5 force-pushed the person/andonova/vdk-job-troubleshooting-plugin branch from 1c7ce4f to 108ac6f Compare December 13, 2022 08:42
@doks5
Copy link
Contributor Author

doks5 commented Dec 13, 2022

Did you use cookiecutter https://github.com/tozka/cookiecutter-vdk-plugin ?

I copied the ingest http plugin and removed all unnecessary stuff. I did not know the cookiecutter plugin existed.

@doks5 doks5 merged commit b73c5cf into main Dec 13, 2022
@doks5 doks5 deleted the person/andonova/vdk-job-troubleshooting-plugin branch December 13, 2022 08:45
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.

None yet

4 participants