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

Role import logging #1959

Merged
merged 13 commits into from
Nov 7, 2023
Merged

Role import logging #1959

merged 13 commits into from
Nov 7, 2023

Conversation

jctanner
Copy link
Contributor

@jctanner jctanner commented Nov 1, 2023

Mimics the collection upload logging architecture in pulp_ansible. Retains compatibility with the CLI ...

(venv) [jtanner@p1 galaxy_ng.role_import_logs]$ ansible-galaxy role import jctanner ansible-role-supertags                                                                                                                                                                                                                    
Successfully submitted import request 2054410600667350699912872248124794885
Starting import: task_id=2054410600667350699912872248124794885, pulp_id=018baa43-4c57-73e2-b61b-96d242966805

==== PARAMETERS ====
importer username: admin
matched user: admin id:1
github_user: jctanner
github_repo: ansible-role-supertags
github_reference: None
alternate_role_name: supertags

==== CHECK FOR MATCHING ROLE(S) ====
user:jctanner repo:ansible-role-supertags matched existing role jctanner.supertags_three id:15

===== CLONING REPO =====
cloning https://github.com/jctanner/ansible-role-supertags ...

===== GIT ATTRIBUTES =====
github_reference(branch): main
github_commit: 41faee088e10474923e15462f1b5f0d608641d3d
github_commit_message: add some tasks
github_commit_date: 2023-11-07T09:50:08-05:00

===== LOADING ROLE =====
Importing with galaxy-importer 0.4.16
Determined role name to be supertags_three
Linting role supertags_three via ansible-lint...
ansible-role-supertags/tests/test.yml:5:7: syntax-check[specific]: the role 'jctanner.supertags' was not found in /tmp/tmpngwagjo5/ansible-role-supertags/tests/roles:/var/lib/pulp/.cache/ansible-compat/eae0d8/roles:/var/lib/pulp/.ansible/roles:/usr/share/ansible/roles:/etc/ansible/roles:/tmp/tmpngwagjo5/ansible-role-supertags/tests
...ansible-lint run complete
Legacy role loading complete

===== PROCESSING LOADER RESULTS ====
enumerated role name supertags_three

===== COMPUTING ROLE VERSIONS ====
tag: v1.0.0 version: 1.0.0
tag: v2.0.0 version: 2.0.0
tag: v3.0.0 version: 3.0.0

==== SAVING ROLE ====

Import completed

@github-actions github-actions bot added backport-4.2 This PR should be backported to stable-4.2 (1.2) backport-4.4 This PR should be backported to stable-4.4 (2.1) backport-4.5 This PR should be backported to stable-4.5 (2.2) backport-4.6 This PR should be backported to stable-4.6 (2.3) backport-4.7 This PR should be backported to stable-4.7 (2.4) backport-4.8 This PR should be backported to stable-4.8 (2.4) labels Nov 1, 2023
@jctanner jctanner removed backport-4.2 This PR should be backported to stable-4.2 (1.2) backport-4.4 This PR should be backported to stable-4.4 (2.1) backport-4.5 This PR should be backported to stable-4.5 (2.2) backport-4.6 This PR should be backported to stable-4.6 (2.3) backport-4.7 This PR should be backported to stable-4.7 (2.4) backport-4.8 This PR should be backported to stable-4.8 (2.4) labels Nov 1, 2023
@jctanner
Copy link
Contributor Author

jctanner commented Nov 1, 2023

@geerlingguy any thoughts on this?

Copy link
Contributor

@bmclaughlin bmclaughlin left a comment

Choose a reason for hiding this comment

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

LGTM once more tests are passing

No-Issue

Signed-off-by: James Tanner <tanner.jc@gmail.com>
No-Issue

Signed-off-by: James Tanner <tanner.jc@gmail.com>
No-Issue

Signed-off-by: James Tanner <tanner.jc@gmail.com>
No-Issue

Signed-off-by: James Tanner <tanner.jc@gmail.com>
No-Issue

Signed-off-by: James Tanner <tanner.jc@gmail.com>
No-Issue

Signed-off-by: James Tanner <tanner.jc@gmail.com>
No-Issue

Signed-off-by: James Tanner <tanner.jc@gmail.com>
No-Issue

Signed-off-by: James Tanner <tanner.jc@gmail.com>
No-Issue

Signed-off-by: James Tanner <tanner.jc@gmail.com>
No-Issue

Signed-off-by: James Tanner <tanner.jc@gmail.com>
No-Issue

Signed-off-by: James Tanner <tanner.jc@gmail.com>
No-Issue

Signed-off-by: James Tanner <tanner.jc@gmail.com>
No-Issue

Signed-off-by: James Tanner <tanner.jc@gmail.com>
@jctanner jctanner merged commit 0f8602e into ansible:master Nov 7, 2023
20 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants