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 notes about use of local users #1335

Merged
merged 4 commits into from
May 31, 2024
Merged

Add notes about use of local users #1335

merged 4 commits into from
May 31, 2024

Conversation

ianf77
Copy link
Contributor

@ianf77 ianf77 commented May 10, 2024

Documentation should clarify if local users are needed

https://issues.redhat.com/browse/AAP-19240

@ianf77 ianf77 added documentation Improvements or additions to documentation Needs backport to 2.4 Changes still need to be applied to the AAP 2.4 branch Controller Content applies to automation controller 2.4 Content applies to AAP 2.4 labels May 10, 2024
@tvo318 tvo318 requested a review from jskacel May 10, 2024 14:05
@tvo318
Copy link
Member

tvo318 commented May 13, 2024

To @jskacel's feedback, we'll just want to change this note to:

[NOTE]

You must have a 'local' user access to configure any of the following:

  • PostgreSQL
  • AWX
  • redis
  • receptor
  • nginx
  • {ControllerName}

@jskacel - please confirm.

Documentation should clarify if local users are needed

https://issues.redhat.com/browse/AAP-19240
Corrected and added details of other user accounts

Documentation should clarify if local users are needed

https://issues.redhat.com/browse/AAP-19240
Updated text

Documentation should clarify if local users are needed

https://issues.redhat.com/browse/AAP-19240
Copy link

@jskacel jskacel left a comment

Choose a reason for hiding this comment

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

LGTM

Correction

Documentation should clarify if local users are needed

https://issues.redhat.com/browse/AAP-19240
Copy link
Collaborator

@hherbly hherbly left a comment

Choose a reason for hiding this comment

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

LGTM

@hherbly hherbly merged commit bc65391 into ansible:main May 31, 2024
ianf77 added a commit to ianf77/aap-docs that referenced this pull request May 31, 2024
* Add notes about use of local users

Documentation should clarify if local users are needed

https://issues.redhat.com/browse/AAP-19240

* Add notes about use of local users

Corrected and added details of other user accounts

Documentation should clarify if local users are needed

https://issues.redhat.com/browse/AAP-19240

* Add notes about use of local users

Updated text

Documentation should clarify if local users are needed

https://issues.redhat.com/browse/AAP-19240

* Added notes about local users.

Correction

Documentation should clarify if local users are needed

https://issues.redhat.com/browse/AAP-19240
hherbly pushed a commit that referenced this pull request May 31, 2024
* Add notes about use of local users

Documentation should clarify if local users are needed

https://issues.redhat.com/browse/AAP-19240

* Add notes about use of local users

Corrected and added details of other user accounts

Documentation should clarify if local users are needed

https://issues.redhat.com/browse/AAP-19240

* Add notes about use of local users

Updated text

Documentation should clarify if local users are needed

https://issues.redhat.com/browse/AAP-19240

* Added notes about local users.

Correction

Documentation should clarify if local users are needed

https://issues.redhat.com/browse/AAP-19240
@ianf77 ianf77 deleted the AAP-19240 branch May 31, 2024 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.4 Content applies to AAP 2.4 Controller Content applies to automation controller documentation Improvements or additions to documentation Needs backport to 2.4 Changes still need to be applied to the AAP 2.4 branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants