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

Add docs validation workflows #21

Merged
merged 1 commit into from
Aug 17, 2022

Conversation

alinabuzachis
Copy link
Collaborator

@alinabuzachis alinabuzachis commented Jul 7, 2022

SUMMARY

Generate diffs and artefacts when someone pushes a docs PR.

ISSUE TYPE
  • Docs Pull Request
COMPONENT NAME

.github

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

ansible-galaxy-importer FAILURE in 3m 36s
✔️ build-ansible-collection SUCCESS in 3m 20s
ansible-test-sanity-docker-devel FAILURE in 8m 55s (non-voting)
ansible-test-sanity-docker-milestone FAILURE in 8m 04s
ansible-test-sanity-docker-stable-2.11 FAILURE in 9m 36s
ansible-test-sanity-docker-stable-2.12 FAILURE in 10m 10s
✔️ ansible-test-integration-amazon-cloud SUCCESS in 2m 26s

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

ansible-galaxy-importer FAILURE in 5m 32s
✔️ build-ansible-collection SUCCESS in 3m 20s
✔️ ansible-test-sanity-docker-devel SUCCESS in 8m 23s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 34s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 8m 21s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 8m 09s
✔️ ansible-test-integration-amazon-cloud SUCCESS in 2m 38s

@alinabuzachis
Copy link
Collaborator Author

recheck

@softwarefactory-project-zuul
Copy link
Contributor

This change depends on a change with an invalid configuration.

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

ansible-galaxy-importer FAILURE in 3m 56s
✔️ build-ansible-collection SUCCESS in 3m 29s
✔️ ansible-test-sanity-docker-devel SUCCESS in 8m 47s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 8m 23s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 7m 59s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 7m 55s
✔️ ansible-test-integration-amazon-cloud SUCCESS in 3m 01s

@alinabuzachis
Copy link
Collaborator Author

alinabuzachis commented Jul 7, 2022

Merged the changes into my fork's main branch and tested with https://github.com/alinabuzachis/amazon.cloud/pull/2/files

[DONE] It seems I have to further clean the module's documentation during the docs generation process and remove non literal strings. Will do that in a PR against the generator repo fixing the doc_fragments imports also.

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

ansible-galaxy-importer FAILURE in 4m 02s
✔️ build-ansible-collection SUCCESS in 3m 33s
✔️ ansible-test-sanity-docker-devel SUCCESS in 9m 26s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 8m 42s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 8m 30s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 59s
✔️ ansible-test-integration-amazon-cloud SUCCESS in 2m 46s

@alinabuzachis alinabuzachis requested a review from gravesm July 26, 2022 09:42
@gravesm
Copy link
Member

gravesm commented Jul 26, 2022

I'm not sure I understand the status of this. Can you explain what is needed to get the CI to pass? I'd strongly prefer we not merge a change that is going to make it so we have to force merge over broken CI on every PR.

@alinabuzachis
Copy link
Collaborator Author

alinabuzachis commented Jul 26, 2022

I'm not sure I understand the status of this. Can you explain what is needed to get the CI to pass? I'd strongly prefer we not merge a change that is going to make it so we have to force merge over broken CI on every PR.

@gravesm I reproduced the same thing tremble did for community.aws ansible/ansible-zuul-jobs#1587 (comment) This doesn't require us to copy doc fragments in the amazon.cloud collection.
Didn't add the galaxy importer things, not sure if we need them now. I can add those if needed.

@gravesm
Copy link
Member

gravesm commented Jul 26, 2022

I think we need to add it, because we can't merge a PR that is going to perpetually break CI.

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 3m 17s
✔️ build-ansible-collection SUCCESS in 3m 19s
✔️ ansible-test-sanity-docker-devel SUCCESS in 9m 00s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 8m 18s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 8m 30s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 6m 53s
✔️ ansible-test-integration-amazon-cloud SUCCESS in 2m 38s

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 3m 56s
✔️ build-ansible-collection SUCCESS in 3m 29s
✔️ ansible-test-sanity-docker-devel SUCCESS in 6m 44s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 7m 42s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 8m 13s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 7m 10s
✔️ ansible-test-integration-amazon-cloud SUCCESS in 2m 41s

@softwarefactory-project-zuul
Copy link
Contributor

Pull request merge failed: Resource not accessible by integration, You may need to manually rebase your PR and retry.

Signed-off-by: Alina Buzachis <abuzachis@redhat.com>

Signed-off-by: Alina Buzachis <abuzachis@redhat.com>
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 3m 46s
✔️ build-ansible-collection SUCCESS in 3m 23s
✔️ ansible-test-sanity-docker-devel SUCCESS in 8m 19s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 6m 50s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 7m 35s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 7m 07s
✔️ ansible-test-integration-amazon-cloud SUCCESS in 2m 42s

@softwarefactory-project-zuul
Copy link
Contributor

Pull request merge failed: Resource not accessible by integration, You may need to manually rebase your PR and retry.

@jillr jillr added mergeit and removed mergeit labels Aug 17, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 05s
✔️ build-ansible-collection SUCCESS in 3m 32s
✔️ ansible-test-sanity-docker-devel SUCCESS in 7m 17s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 8m 27s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 7m 52s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 7m 40s
✔️ ansible-test-integration-amazon-cloud SUCCESS in 2m 50s

@softwarefactory-project-zuul
Copy link
Contributor

Pull request merge failed: Resource not accessible by integration, You may need to manually rebase your PR and retry.

@gravesm gravesm merged commit 273cd55 into ansible-collections:main Aug 17, 2022
@github-actions
Copy link

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and your docs changes will be incorporated when they are next published.

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