Skip to content

Conversation

@laem
Copy link

@laem laem commented Aug 19, 2025

Je ne sais pas à quel point il faut merger cette PR directement. Je ne suis pas expert ansible, je pense avoir corrigé beaucoup de choses qui ont évolué dans les dernières versions d'overpass, mais j'ai aussi peut-être simplifié des choses qui pourraient être utiles.

Aussi, je n'ai pas encore lancé le sujet areas. Il faut tester et MAJ si ça marche pas.

jocelynj

This comment was marked as duplicate.

@@ -1,471 +1,2 @@
osm11.openstreetmap.fr
Copy link
Member

Choose a reason for hiding this comment

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

Commit à supprimer ?

Est-ce que tu aurais encore l’erreur toml liée ?

@@ -1,7 +1,10 @@
- name: overpass-api.yml
hosts: overpass-api
hosts: maelito
Copy link
Member

Choose a reason for hiding this comment

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

-> overpass-api

Et mettre ton host dans la section overpass-api.

strip: 1
become: yes
become_user: "{{ overpass_user }}"
when: false # Looks to me that the diff file used to patch used to target another script and fails in the following versions. Disabling it an hoping it was fixed / not so important :)
Copy link
Member

Choose a reason for hiding this comment

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

On peut supprimer cette task du coup.


log.info(url)
log.info(obj_datetime)

Copy link
Member

Choose a reason for hiding this comment

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

Commit 8da034b à supprimer maintenant ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants