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

Fix validation errors in plugin documentation #399

Conversation

gravesm
Copy link
Member

@gravesm gravesm commented Mar 7, 2022

Depends-On: ansible/ansible-zuul-jobs#1385

SUMMARY

This fixes validation errors in plugin documentation now that ansible
test for 2.13 is running validate-modules on all plugins. The kubectl
connection plugin validation is ignored because there seems to be a
requirement for the author field to have a github username, which we do
not have.

ISSUE TYPE
  • Docs Pull Request
COMPONENT NAME

plugins/connection/kubectl
plugins/inventory/k8s
plugins/lookup/k8s
plugins/lookup/kustomize

ADDITIONAL INFORMATION

@softwarefactory-project-zuul
Copy link

Build failed.

@Akasurde Akasurde force-pushed the plugin-validation branch from b9b4115 to d56f568 Compare March 8, 2022 07:14
@softwarefactory-project-zuul
Copy link

Build failed.

This fixes validation errors in plugin documentation now that ansible
test for 2.13 is running validate-modules on all plugins. The kubectl
connection plugin validation is ignored because there seems to be a
requirement for the author field to have a github username, which we do
not have.

Signed-off-by: Abhijeet Kasurde <[email protected]>
@Akasurde Akasurde force-pushed the plugin-validation branch from d56f568 to 5893d9e Compare March 8, 2022 09:16
@softwarefactory-project-zuul
Copy link

Build failed.

@gravesm
Copy link
Member Author

gravesm commented Mar 8, 2022

recheck

@softwarefactory-project-zuul
Copy link

Build succeeded.

@abikouo
Copy link
Contributor

abikouo commented Mar 8, 2022

recheck

@softwarefactory-project-zuul
Copy link

Build failed.

@goneri
Copy link
Member

goneri commented Mar 10, 2022

recheck

@softwarefactory-project-zuul
Copy link

Build failed.

@abikouo
Copy link
Contributor

abikouo commented Mar 11, 2022

recheck

@softwarefactory-project-zuul
Copy link

Build succeeded.

@softwarefactory-project-zuul
Copy link

Build succeeded (gate pipeline).

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 30e84fa into ansible-collections:main Mar 11, 2022
@gravesm gravesm deleted the plugin-validation branch March 11, 2022 18:27
gravesm added a commit to gravesm/kubernetes.core that referenced this pull request May 12, 2022
Fix validation errors in plugin documentation

Depends-On: ansible/ansible-zuul-jobs#1385
SUMMARY

This fixes validation errors in plugin documentation now that ansible
test for 2.13 is running validate-modules on all plugins. The kubectl
connection plugin validation is ignored because there seems to be a
requirement for the author field to have a github username, which we do
not have.

ISSUE TYPE

Docs Pull Request

COMPONENT NAME

plugins/connection/kubectl
plugins/inventory/k8s
plugins/lookup/k8s
plugins/lookup/kustomize
ADDITIONAL INFORMATION

Reviewed-by: None <None>
(cherry picked from commit 30e84fa)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request May 12, 2022
[backport/2.2] Fix validation errors in plugin documentation (#399)

Fix validation errors in plugin documentation
SUMMARY
This fixes validation errors in plugin documentation now that ansible
test for 2.13 is running validate-modules on all plugins. The kubectl
connection plugin validation is ignored because there seems to be a
requirement for the author field to have a github username, which we do
not have.
ISSUE TYPE
Docs Pull Request
COMPONENT NAME
plugins/connection/kubectl
plugins/inventory/k8s
plugins/lookup/k8s
plugins/lookup/kustomize
ADDITIONAL INFORMATION
Reviewed-by: None 
(cherry picked from commit 30e84fa)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants