Commit graph

25 commits

Author SHA1 Message Date
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
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
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
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
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
c349a7c25e chrony: try and fix chrony in staging
In staging we set a 'staging' datacenter, but this makes the chrony
template think they are external hosts not in iad2, which is wrong.
So, add a conditional to make them use our internal iad2 ntp servers if
they are in staging datacenter too.

Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2021-02-05 12:24:03 -08:00
Kevin Fenzi
7984b46eb7 The great phx2 pruning run (1st cut).
Since we no longer have any machines in phx2, I have tried to remove
them from ansible. Note that there are still some places where we need
to remove them still: nagios, dhcp, named were not touched, and in cases
where it wasn't pretty clear what a conditional was doing I left it to
be cleaned up later.

Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2020-06-14 14:14:31 -07:00
Kevin Fenzi
e6784105c6 chrony: make whitespace change in iad2 to check restarting
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2020-05-16 12:02:10 -07:00
Kevin Fenzi
9f6463ce8f chrony: add handler to restart on config changes
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2020-05-16 12:01:22 -07:00
Kevin Fenzi
855b87b3d7 chrony: fix template to work in iad2
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2020-05-16 11:41:48 -07:00
Kevin Fenzi
5432763f39 chrony: drop task to stop/disable ntp, we will not even install it anymore
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2020-05-16 10:40:43 -07:00
Stephen Smoogen
efcd9b0ead the problem was that some of the host names were listed as cloud-noc01.fedorainfracloud.org and others were named cloud-noc01.cloud.fedoraproject.org. Move to 1 name throughout ansible 2020-04-24 21:34:27 +02:00
Kevin Fenzi
a83eced0d7 chronyd: also allow new cloud network to sync time on cloud-noc01
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2020-04-24 21:34:21 +02:00
Kevin Fenzi
7a306aaa51 chrony: for now ignore errors in stopping ntp, probibly should just remove it.
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2020-04-24 21:34:14 +02:00
Karsten Hopp
c9ed62ac32 update ansible_distribution_major_version conditionals
Signed-off-by: Karsten Hopp <karsten@redhat.com>
2020-04-24 21:34:10 +02:00
Mikolaj Izdebski
606152627d Revert "chrony: Don't disable and stop ntp on Fedora >= 31"
This reverts commit aad9ee4ca340b7255462fab5bdaf3eb8536b66f1.
2020-04-24 21:34:10 +02:00
Mikolaj Izdebski
a7308bc667 chrony: Don't disable and stop ntp on Fedora >= 31 2020-04-24 21:34:10 +02:00
Kevin Fenzi
57d7b73581 chrony: Make sure cloud-noc01 listens on port 124 for requests
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2019-05-08 22:44:34 +00:00
Kevin Fenzi
c876bb1fed chrony: ntp doesn't exist in rhel8beta so we don't try and stop and disable it.
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2019-02-12 22:18:59 +00:00
Kevin Fenzi
086296b296 this is a /8 2018-09-27 00:55:25 +00:00
Kevin Fenzi
b5c368da1e need bastions to be servers too 2018-09-27 00:52:50 +00:00
Kevin Fenzi
6bdc749a76 fix mistag 2018-09-26 23:19:20 +00:00
Kevin Fenzi
3a66476c03 I give myself a D for the last 2 commits 2018-09-26 20:49:39 +00:00
Kevin Fenzi
ae01fea545 this is ntpd here 2018-09-26 20:48:24 +00:00
Kevin Fenzi
0258bf95c7 out with the ntp, in with the chrony 2018-09-26 20:44:52 +00:00