Commit Graph

111 Commits

Author SHA1 Message Date
Kevin Fenzi
c567d80f3f ipa / client: block paguremirroring user in ipa
We want to use a local version of this user, not the ipa one.

Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2025-11-21 12:51:46 -08:00
Kevin Fenzi
17fb5e7e3a ipa / client: filter the git user
We want to use the local git user, not the ipa one on pagure.

Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2025-11-18 18:30:06 -08:00
Kevin Fenzi
888d5632ea ipa / client: stop removing the 04-ipa.conf ssh config
We removed this because it had some options we didn't like long ago.
However, now it's basically all stuff thats already in our base ssh
config.

It causes anoying, but harmless scriptlet messages on updates
and there's no need to remove it to only have it get re-added by
freeipa-client every update.

Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2025-11-14 12:34:23 -08:00
Kevin Fenzi
e57e3dd7e4 ipa/client: mode should be 640 on fedora instances
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2025-08-08 12:29:13 -07:00
Kevin Fenzi
db0e5eb546 ipa / client: fedora systems use root:sssd ownership
On rhel the sssd dropin files (which need to be the same owner/group as
the main sssd.conf file) are root:root, but on fedora they are
root:sssd. So, split out this task to handle the two different cases.

Assisted-by: claude
(I had it generate this, and it actually did a reasonable job I think)

Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2025-08-08 11:54:33 -07:00
Kevin Fenzi
1b67cfcf3b releng-compose: filter some more users that should be local
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2025-07-03 13:58:56 -07:00
Nils Philippsen
6c85fda0c9 Mass remove/replace iad2 -> rdu3, 10.3. -> 10.16.
Signed-off-by: Nils Philippsen <nils@redhat.com>
2025-07-03 20:05:02 +02:00
Kevin Fenzi
90ed0a38e0 pkgs: change the pagure user to uid 1000 for suexec, block in sssd
The pagure user needs to be uid 1000 because suexec won't let users with
uid under that suexec. ;(

Also, filter pagure user out in sssd so we get the local user.

Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2025-07-02 15:25:17 -07:00
Aurélien Bompard
2e243f0b28 SSSd wants its conf snippets to have the same perms as sssd.conf
Signed-off-by: Aurélien Bompard <aurelien@bompard.org>
2025-06-24 17:38:45 +02:00
David Kirwan
809c90e5da IPA: add user zabbix to fedora-nss-ignore.conf
Signed-off-by: David Kirwan <davidkirwanirl@gmail.com>
2025-03-21 16:25:30 +00:00
Kevin Fenzi
d3975febbe ipa/client: sssd drop in needs to be same permission as sssd.conf also
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2025-02-16 14:35:32 -08:00
Kevin Fenzi
258fa9fd14 ipa/client: sssd drop in needs to be owned by root, sssd changes it on restart
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2025-02-16 10:31:28 -08:00
Michal Konecny
2ec055db6f Use first uppercase letter for all handlers
This will unify all the handlers to use first uppercase letter for
ansible-lint to stop complaining.

I went through all `notify:` occurrences and fixed them by running
```
set TEXT "text_to_replace"; set REPLACEMENT "replacement_text"; git grep
-rlz "$TEXT" . | xargs -0 sed -i "s/$TEXT/$REPLACEMENT/g"
```

Then I went through all the changes and removed the ones that wasn't
expected to be changed.

Fixes https://pagure.io/fedora-infrastructure/issue/12391

Signed-off-by: Michal Konecny <mkonecny@redhat.com>
2025-02-10 20:31:49 +00:00
Kevin Fenzi
9af79d19ee handlers: fix another name change
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2025-01-24 14:11:11 -08:00
Kevin Fenzi
13266214d2 ipa / handlers: Fix call to 'restart sssd' that is now 'Restart sssd'
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2025-01-15 16:36:11 -08:00
Ryan Lerch
47c68f478d ansiblelint fixes - fqcn[action-core] - template to ansible.builtin.template
Replaces references to template: with ansible.builtin.template

Signed-off-by: Ryan Lerch <rlerch@redhat.com>
2025-01-15 11:30:29 +10:00
Ryan Lerch
3c41882bb0 ansiblelint fixes - fqcn[action-core] - shell to ansible.builtin.shell
Replaces references to shell: with ansible.builtin.shell

Signed-off-by: Ryan Lerch <rlerch@redhat.com>
2025-01-15 11:29:10 +10:00
Ryan Lerch
25391e95b7 ansiblelint fixes - fqcn[action-core] - package to ansible.builtin.package
Replaces many references to  package: with ansible.builtin.package

Signed-off-by: Ryan Lerch <rlerch@redhat.com>
2025-01-15 11:28:00 +10:00
Ryan Lerch
462176464b ansiblelint fixes-- fqcn[action-core] - command to ansible.builtin.command
Replaces many references to  command: with ansible.builtin.command

Signed-off-by: Ryan Lerch <rlerch@redhat.com>
2025-01-15 11:26:47 +10:00
Ryan Lerch
62952df107 ansiblelint fixes-- fqcn[action-core] - file to ansible.builtin.file
Replaces many references to  file: with ansible.builtin.file

Signed-off-by: Ryan Lerch <rlerch@redhat.com>
2025-01-15 10:41:52 +10:00
Ryan Lerch
691adee6ee Fix name[casing] ansible-lint issues
fix 1900 failures of the following case issue:

`name[casing]: All names should start with an uppercase letter.`

Signed-off-by: Ryan Lerch <rlerch@redhat.com>
2025-01-14 20:20:07 +10:00
Kevin Fenzi
ce1f5b02e6 ipa_client: on f40 there is no sssd user, so files are owned by root
On rhel and f41+ there is a sssd user, so we should use that.
If we don't, sssd will change the ownership on restart, meaning we flip
it back and forth each time we run the playbook.

remember to remove this when fedora 40 is all gone from infra

Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2024-12-10 14:43:47 -08:00
Kevin Fenzi
df36530d00 ipa_client: add tag for nss ignore file to allow globally updating it.
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2024-12-10 14:01:54 -08:00
Kevin Fenzi
aaa29839fa ipa_client: the fedora-sss-ignore.conf file should be owned by sssd user/group
We change this to root/root and then restart sssd and it changes it
back. So, lets do this right and let it be sssd/sssd.

Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2024-12-09 17:54:13 -08:00
Ryan Lerch
89f6f1fc32 Fix majority of remaining yamllint warnings and errors
Signed-off-by: Ryan Lerch <rlerch@redhat.com>
2024-11-28 17:31:45 +10:00
Kevin Fenzi
3a2623218d ipa client: filer out mysql user from ipa/ldap
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2024-11-20 16:48:40 -08:00
Kevin Fenzi
aa5c7bac75 ipa / client / sssd: add rawhide to filter and fix syntax error
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2024-07-17 18:05:41 -07:00
Pavel Raiskup
23275c26ea Revert "copr-hypervisor/ipa-client: debug variable"
This reverts commit 9daef0d257.
2024-02-13 00:04:20 +01:00
Pavel Raiskup
9daef0d257 copr-hypervisor/ipa-client: debug variable 2024-02-12 23:55:16 +01:00
Kevin Fenzi
1c43429266 ipa / client: just remove the sshd override file
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2023-09-20 12:10:32 -07:00
Kevin Fenzi
308ff1a8ce Revert "ipa / client: pass --no-sshd to client enroll"
This reverts commit df1445a64b.

Turns out we do have to enable sshd on client enroll because it passes
'ssh' to services in sssd.conf, which we need to get ssh keys for users.
:( Instead will try another approach.
2023-09-20 12:07:06 -07:00
Kevin Fenzi
df1445a64b ipa / client: pass --no-sshd to client enroll
In RHEL9, ipa-enroll-client by default adds a
/etc/ssh/sshd_config.d/04-ipa.conf file with some sshd configuration.
Almost all of these things are things we already set in our sshd_config,
but one of them causes sshd to enable password (and 2nd factor required)
auth. We don't want this, we only want to allow ssh keys.
So, pass --no-sshd to enrollment and that should prevent it from
messing with our sshd config.

I have also removed this file and reloaded sshd all around.

Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2023-09-20 10:52:17 -07:00
Kevin Fenzi
cf5ca54701 ipa: exclude the ipa postgres user in favor of local one
We have a postgres user in ipa to prevent people from making an account
with that name, but we need/want to use a local version of this on
database servers, not the ipa one. We need the local one because the ipa
one is locked and this prevents database backups from working.
(Locked accounts can't run cron jobs).

Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2023-06-07 16:52:25 -07:00
Kevin Fenzi
6720b82d01 ipa/client: try and sort this so it does not change from run to run
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2023-04-04 13:03:37 -07:00
Kevin Fenzi
4df0d9e28f ipa/client: hbac check Set this to not be changed since it's a check
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2023-04-04 13:02:46 -07:00
Kevin Fenzi
25484cc0a1 ipa / hbac: lower case sudo group
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2023-02-14 12:24:19 -08:00
Kevin Fenzi
e2a5be9575 put nolog back
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2023-02-13 15:39:35 -08:00
Kevin Fenzi
d943c375e6 temp disable nolog here to gather debugging info
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2023-02-13 15:21:40 -08:00
Kevin Fenzi
e42b099606 ipa / client: try not double delegating
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2023-02-13 15:12:51 -08:00
Kevin Fenzi
e2fdd2826a temp disable nolog here to gather debugging info
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2023-02-13 15:00:22 -08:00
Kevin Fenzi
527d8cda18 sssd: exclude rabbitmq user also
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2022-10-03 16:12:55 -07:00
Kevin Fenzi
3c960624f4 fas2: good bye! You served long and well
Remove fas2 and all the checks that depended on it.

Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2022-07-01 12:09:36 -07:00
Kevin Fenzi
35a977170e ipa: set nolog back on
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2022-05-19 17:11:23 -07:00
Kevin Fenzi
a8851731d0 Revert "Revert "ipa/client: temp remove nolog""
This reverts commit 04bd033a9b.
2022-05-19 11:05:34 -07:00
Kevin Fenzi
04bd033a9b Revert "ipa/client: temp remove nolog"
This reverts commit 6bb9f0a4ea.
2022-05-19 11:00:04 -07:00
Kevin Fenzi
6bb9f0a4ea ipa/client: temp remove nolog
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2022-05-19 10:58:23 -07:00
Kevin Fenzi
51e1424f5d ipa/client: add a debug for ipa_servers
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2022-05-19 10:01:19 -07:00
Kevin Fenzi
1a069052f0 ipa/client: add mirrormanager user/group to ipa excludes
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2021-07-16 08:15:15 -07:00
František Zatloukal
9f273622e0 Blockerbugs: force to use local user instead of the ipa one 2021-06-09 19:14:13 +02:00
Kevin Fenzi
48e22151ae ipa/client: flush handlers at the end of ipa/client
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2021-05-24 15:24:40 -07:00