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

common: ansible_user support in ansible scripts #6055

Merged
merged 3 commits into from
Oct 3, 2024

Conversation

grom72
Copy link
Contributor

@grom72 grom72 commented Mar 14, 2024

This PR is a replacement for #5955.


This change is Reviewable

@grom72 grom72 added sprint goal This pull request is part of the ongoing sprint no changelog Add to skip the changelog check on your pull request labels Mar 14, 2024
@grom72 grom72 requested a review from janekmi March 14, 2024 15:51
@grom72 grom72 requested a review from osalyk March 14, 2024 15:51
Copy link

codecov bot commented Mar 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.07%. Comparing base (9819db6) to head (9316218).
Report is 121 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #6055   +/-   ##
=======================================
  Coverage   70.07%   70.07%           
=======================================
  Files         133      133           
  Lines       19563    19563           
  Branches     3261     3261           
=======================================
  Hits        13708    13708           
  Misses       5855     5855           

@janekmi janekmi added this to the 2.1.0 milestone Mar 14, 2024
@grom72 grom72 added build system What Makefiles do and removed sprint goal This pull request is part of the ongoing sprint labels Mar 15, 2024
Copy link
Contributor

@osalyk osalyk left a comment

Choose a reason for hiding this comment

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

Reviewed 2 of 3 files at r1, 1 of 3 files at r2, 3 of 3 files at r3, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @janekmi)

@grom72 grom72 modified the milestones: 2.1.0, 2.1.1 Mar 27, 2024
@janekmi janekmi modified the milestones: 2.1.1, 2.1.0 Mar 27, 2024
@grom72 grom72 modified the milestones: 2.1.0, 2.x Mar 28, 2024
@grom72 grom72 added the sprint goal This pull request is part of the ongoing sprint label Sep 16, 2024
Copy link
Contributor

@janekmi janekmi left a comment

Choose a reason for hiding this comment

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

Reviewed 3 of 3 files at r1, 3 of 3 files at r2, 3 of 3 files at r3, all commit messages.
Reviewable status: all files reviewed, 5 unresolved discussions (waiting on @grom72)


utils/ansible/opensuse-setup.yml line 9 at r2 (raw file):

# in the README.md file.
#
# Below is an example of how to use this file localy:

Suggestion:

Below is an example of how to use this file locally:

utils/ansible/opensuse-setup.yml line 12 at r2 (raw file):

#
# a) comment out the first command:
# -hosts: "{{ host }}"

Suggestion:

# - hosts: "{{ host }}"

utils/ansible/rockylinux-setup.yml line 9 at r2 (raw file):

# in the README.md file.
#
# Below is an example of how to use this file localy:

Suggestion:

Below is an example of how to use this file locally:

utils/ansible/rockylinux-setup.yml line 12 at r2 (raw file):

#
# a) comment out the first command:
# -hosts: "{{ host }}"

Suggestion:

# - hosts: "{{ host }}"

utils/ansible/update-os.yml line 10 at r3 (raw file):

#
# Note: This playbook is designed to work as a composite action during test preparation steps.
# Local usage might cause imidiate system reboot.

Suggestion:

Local usage might cause an immediate system reboot.

@grom72 grom72 force-pushed the local_ansible_user branch 2 times, most recently from e30ff28 to 8930e1d Compare October 1, 2024 06:00
Copy link
Contributor Author

@grom72 grom72 left a comment

Choose a reason for hiding this comment

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

Reviewable status: 0 of 6 files reviewed, 5 unresolved discussions (waiting on @janekmi and @osalyk)


utils/ansible/opensuse-setup.yml line 9 at r2 (raw file):

# in the README.md file.
#
# Below is an example of how to use this file localy:

Done.


utils/ansible/opensuse-setup.yml line 12 at r2 (raw file):

#
# a) comment out the first command:
# -hosts: "{{ host }}"

Done.


utils/ansible/rockylinux-setup.yml line 9 at r2 (raw file):

# in the README.md file.
#
# Below is an example of how to use this file localy:

Done.


utils/ansible/rockylinux-setup.yml line 12 at r2 (raw file):

#
# a) comment out the first command:
# -hosts: "{{ host }}"

Done.


utils/ansible/update-os.yml line 10 at r3 (raw file):

#
# Note: This playbook is designed to work as a composite action during test preparation steps.
# Local usage might cause imidiate system reboot.

Done.

Copy link
Contributor

@janekmi janekmi left a comment

Choose a reason for hiding this comment

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

Reviewed 6 of 6 files at r5, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @grom72)


utils/ansible/update-os.yml line 10 at r5 (raw file):

#
# Note: This playbook is designed to work as a composite action during test preparation steps.
# Local usage might cause an immediate system reboot.

Nitpick but since you divided this change into three commits I believe this bit should be a part of the "common: ansible_user support in ansible scripts" rather than "common: ansible_user support in ansible scripts". Please fix.

@grom72 grom72 force-pushed the local_ansible_user branch 2 times, most recently from f5c3f6d to 2258d3e Compare October 2, 2024 13:47
Copy link
Contributor Author

@grom72 grom72 left a comment

Choose a reason for hiding this comment

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

Reviewed 5 of 6 files at r6, 1 of 3 files at r7, 5 of 5 files at r8, 6 of 6 files at r9, 3 of 3 files at r10.
Reviewable status: 1 of 6 files reviewed, 1 unresolved discussion (waiting on @janekmi)


utils/ansible/update-os.yml line 10 at r5 (raw file):

Previously, janekmi (Jan Michalski) wrote…

Nitpick but since you divided this change into three commits I believe this bit should be a part of the "common: ansible_user support in ansible scripts" rather than "common: ansible_user support in ansible scripts". Please fix.

Done.

Copy link
Contributor Author

@grom72 grom72 left a comment

Choose a reason for hiding this comment

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

Reviewed 5 of 5 files at r11, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @janekmi)

Copy link
Contributor Author

@grom72 grom72 left a comment

Choose a reason for hiding this comment

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

Reviewed 1 of 6 files at r5.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @janekmi)

Copy link
Contributor

@janekmi janekmi left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 1 of 3 files at r10, 5 of 5 files at r11, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @grom72)

@janekmi janekmi merged commit ac81f4c into pmem:master Oct 3, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build system What Makefiles do no changelog Add to skip the changelog check on your pull request sprint goal This pull request is part of the ongoing sprint
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants