Commit graph

71 commits

Author SHA1 Message Date
Slavi Pantaleev 65bfc1396e Revert "Update Hookshot 1.8.0 -> 1.8.1"
This reverts commit 05ccee9f6f.

Fixes https://github.com/spantaleev/matrix-docker-ansible-deploy/issues/1952
2022-07-19 16:47:08 +03:00
Aine 05ccee9f6f
Update Hookshot 1.8.0 -> 1.8.1
No docker tag published yet, [keep an eye on it](https://hub.docker.com/r/halfshot/matrix-hookshot/tags?page=1&ordering=last_updated&name=1.8.1)
2022-07-18 15:37:04 +00:00
Slavi Pantaleev c73680712b Fix (suppress) var-naming ansible-lint errors
Reference: https://ansible-lint.readthedocs.io/en/latest/default_rules/#var-naming

We don't really fix these, but just suppress them,
because they're like that intentionally.

We try to name variables in a way that is consistent with the
configuration key they control. If the upstream component uses
camelCase, we also need to include camelCase in the variable name.
2022-07-18 16:43:12 +03:00
Slavi Pantaleev 1693c4ca1d Fix no-changed-when ansible-lint errors
Reference: https://ansible-lint.readthedocs.io/en/latest/default_rules/#no-changed-when
2022-07-18 15:08:10 +03:00
Slavi Pantaleev d073c7ecb3 More ansible-lint fixes 2022-07-18 13:01:19 +03:00
Slavi Pantaleev ddf18eadc7 More ansible-lint fixes 2022-07-18 13:01:17 +03:00
Slavi Pantaleev 34cdaade08 Use fully-qualified module names for builtin Ansible modules
Related to https://github.com/spantaleev/matrix-docker-ansible-deploy/pull/1939
2022-07-18 12:58:41 +03:00
Aine e149f33140
add/unify 'Project source code URL' link across all roles 2022-07-16 23:59:21 +03:00
Slavi Pantaleev 3b85a9a994
Upgrade hookshot (1.7.3 -> 1.8.0) 2022-07-11 20:25:32 +03:00
Slavi Pantaleev ba51997f7b (BC Break) Redo how metrics are exposed to external Prometheus servers 2022-06-23 17:55:07 +03:00
Slavi Pantaleev 99c24ef0e8 Upgrade Hookshot (1.7.2 -> 1.7.3) 2022-06-09 20:52:56 +03:00
Slavi Pantaleev 12b28a4749 Upgrade Hookshot (1.7.0 -> 1.7.2) 2022-06-09 14:13:53 +03:00
Slavi Pantaleev 677a2fc503 Fix compatibility with ansible=6 / ansible-core=2.13
Details here: https://docs.ansible.com/ansible/devel/porting_guides/porting_guide_6.html#id36

Basically:

```yaml
- name: Prior to 2.13
  debug:
    msg: '[1] + {{ [2] }}'

- name: 2.13 and forward
  debug:
    msg: '{{ [1] + [2] }}'
```

Interestingly, we had been using the new/safe syntax in lofs of places.

We were using the broken one in many others though. Hopefully all
instances were fixed by this patch.
2022-05-18 15:43:39 +03:00
Slavi Pantaleev a1e5ecf551 Upgrade hookshot (1.6.1 -> 1.7.0)
This new version should be buildable on arm64.

See: https://github.com/matrix-org/matrix-hookshot/releases/tag/1.7.0

There's still no prebuild arm64 image, so we continue relying on
self-building there.
2022-05-18 14:43:16 +03:00
Slavi Pantaleev e62123ac13
Merge pull request #1804 from HarHarLinks/feeds
Add hookshot feeds config support
2022-05-08 07:10:40 +02:00
HarHarLinks 7390646cd5 add hookshot feeds support 2022-05-07 18:42:15 +02:00
Kim Brose 84ea5f6ecc
Upgrade Hookshot (1.5.0 -> 1.6.1) 2022-05-07 14:34:33 +02:00
Kim Brose 7e062328ff
Upgrade Hookshot (1.4.0 -> 1.5.0) 2022-04-14 15:01:40 +02:00
Slavi Pantaleev 2df993977a Ensure git cloning when self-building is done with the matrix user, not root
Fixes https://github.com/spantaleev/matrix-docker-ansible-deploy/issues/1749
2022-04-14 08:52:37 +03:00
HarHarLinks a9e6538ef8 Upgrade Hookshot (1.3.0 -> 1.4.0)
https://github.com/matrix-org/matrix-hookshot/releases/tag/1.4.0
2022-04-11 18:55:02 +02:00
HarHarLinks ceb2f54970 Make hookshot hardcoded public URLs dynamic 2022-04-11 16:45:33 +02:00
Luke 627333d82b
fix container image 2022-04-05 12:42:38 +00:00
Arthur Brugière cfd8a9c0f8
[HOOKSHOT] Fix yamllint 2022-04-03 17:19:35 +07:00
Arthur Brugière b3176957c3
Add hookshot self build for arm64 and amd64 2022-04-03 17:10:46 +07:00
Kim Brose ea358e208c
Upgrade Hookshot (1.2.0 -> 1.3.0) 2022-03-30 14:18:21 +02:00
Aine 2da3768b20
Added retries to the docker pulls (#1701) 2022-03-17 17:37:11 +02:00
Jim Myhrberg eeca3c8dca
fix: avoid yaml being wrapped at column 80 via to_nice_yaml
The `to_nice_yaml` helper will by default wrap any string YAML values on
the first space after column 80. This can in worst case yield invalid
YAML syntax. More details in Ansible's documentation here:

https://docs.ansible.com/ansible/latest/user_guide/playbooks_filters.html#formatting-data-yaml-and-json

In short, you need to explicitly provide a custom width argument of a
high number of some kind to avoid the line wrapping.
2022-03-16 01:10:26 +00:00
László Várady 9cc3c5be76 Fix warning about reverse-proxying when built-in proxy is disabled
`matrix_nginx_proxy_enabled` is usually set to false by users, this is when
the warning should be displayed.
2022-03-12 16:29:20 +01:00
Luis 833ac506fe
pass GET parameters too
This fixes parameters not being passed to the provisioning api
2022-03-04 15:08:16 -03:00
Luis 584e50d117
fix hookshot provisioning url in nginx
Hello, this PR should fix Hookshot container not receiving API calls with the right path because nginx proxy was stripping the /v1 from it.
2022-03-04 14:40:05 -03:00
Kim Brose f62279581d
Upgrade Hookshot (1.1.0 -> 1.2.0)
all features were actually already silently supported 🎉

upstream changelog: https://github.com/matrix-org/matrix-hookshot/releases/tag/1.2.0
2022-03-04 16:54:17 +01:00
Slavi Pantaleev fb700743f9
Merge pull request #1635 from HarHarLinks/hookshot-urlprefix
Update hookshot to respect protocol and custom ports
2022-03-04 07:44:01 +02:00
Peter Schümann 211f05abbe add $is_args$args
to proxy url params in GET requests
2022-03-03 22:21:21 +01:00
Slavi Pantaleev 35df276487
Merge pull request #1624 from HarHarLinks/hookshot
update hookshot registration.yml
2022-03-03 09:37:45 +02:00
HarHarLinks 58771a9c65 hookshot moved to matrix-org namespace 2022-03-02 20:58:11 +01:00
HarHarLinks d556952bb6 update hookshot to respect protocol and custom ports 2022-02-16 20:10:01 +01:00
Slavi Pantaleev 1ed46f0a86 Remove Jinja2 templating delimiters from when statement
Fixes https://github.com/spantaleev/matrix-docker-ansible-deploy/issues/1629

Related to https://github.com/spantaleev/matrix-docker-ansible-deploy/pull/1505
2022-02-16 09:36:33 +02:00
HarHarLinks 735eec92cc update hookshot registration.yml
based on comments in chat:
https://matrix.to/#/!TlZdPIYrhwNvXlBiEk:half-shot.uk/$RbG6itEHVV8J_u5ry1HiFdC76n19M3vmopfQOyRmkKU?via=half-shot.uk&via=matrix.org&via=envs.net
2022-02-13 17:31:52 +01:00
Marko Weltzer 819574b8ba
Merge branch 'spantaleev:master' into master 2022-02-05 21:37:53 +01:00
Marko Weltzer 7e5b88c3b7 fix: all praise the allmighty yamllinter 2022-02-05 21:32:54 +01:00
Slavi Pantaleev 86c36523df Replace ExecStopPost with ExecStop
Reverts b1b4ba501f, 90c9801c56, a3c84f78ca, ..

I haven't really traced it (yet), but on some servers, I'm observing
`ansible-playbook ... --tags=start` completing very slowly, waiting
to stop services. I can't reproduce this on all Matrix servers I manage.
I suspect that either the systemd version is to blame or that some
specific service is not responding well to some `docker kill/rm` command.

`ExecStop` seems to work great in all cases and it's what we've been
using for a very long time, so I'm reverting to that.
2022-02-05 12:13:36 +02:00
HarHarLinks 51baa40eff fixup! escape hookshot variables using to_json 2022-01-30 19:28:40 +01:00
HarHarLinks 99cf6adf95 fixup! update hookshot private key installation method 2022-01-30 19:21:26 +01:00
HarHarLinks 5300dc7a8b update hookshot/defaults/main.yml comments 2022-01-30 19:05:49 +01:00
HarHarLinks f136c1fb8f fixup! add some hookshot config validation 2022-01-30 18:53:57 +01:00
HarHarLinks 4477711e79 add some hookshot config validation 2022-01-30 17:46:40 +01:00
HarHarLinks f47bfdfb4f remove hookshot uninstall comments based on feedback 2022-01-30 17:46:01 +01:00
HarHarLinks 29d0b277f4 fix hookshot missing figma config variable 2022-01-30 17:45:28 +01:00
HarHarLinks a3c84f78ca update hookshot.service to ExecStopPost 2022-01-30 17:45:09 +01:00
HarHarLinks 99802bc143 escape hookshot variables using to_json 2022-01-30 17:44:22 +01:00