entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 34 Releases Activity

misc: do not alter base_url in stored hobo.json (#21605) #832

Merged
fpeters merged 1 commits from wip/21605-keep-hobo-json-as-is into main 2023-11-24 16:15:23 +01:00
Conversation 0 Commits 1 Files Changed 2 +6 -6
fpeters commented 2023-11-11 14:24:04 +01:00
Owner
Copy Link
No description provided.
fpeters force-pushed wip/21605-keep-hobo-json-as-is from bd2cb6ece6 to fa9eee5a2b 2023-11-11 14:54:27 +01:00 Compare
fpeters force-pushed wip/21605-keep-hobo-json-as-is from fa9eee5a2b to a12e41d228 2023-11-11 16:57:58 +01:00 Compare
fpeters reviewed 2023-11-11 17:24:11 +01:00
tests/test_hobo.py
@ -256,2 +256,3 @@
hobo_json['services'][2]['base_url'] = 'http://wcs.example.net/' # reset
del hobo_json['services'][1] # authentic
fd.write(json.dumps(HOBO_JSON))
fd.write(json.dumps(hobo_json))
fpeters commented 2023-11-11 17:22:30 +01:00
Author
Owner
Copy Link

Il y a des tests qui modifient le HOBO_JSON global, je modifie ici pour être sûr d'avoir toujours le bon base_url.

Il y a des tests qui modifient le HOBO_JSON global, je modifie ici pour être sûr d'avoir toujours le bon base_url.
tests/test_hobo.py
@ -624,2 +625,2 @@
'http://wcs.example.net',
'http://wcs2.example.net',
'http://wcs.example.net/',
'http://wcs2.example.net/',
fpeters commented 2023-11-11 17:22:56 +01:00
Author
Owner
Copy Link

La conséquence du patch est qu'on a les / finaux dans les hobo.json stockés, qui sont utilisés pour le --redeploy.

La conséquence du patch est qu'on a les / finaux dans les hobo.json stockés, qui sont utilisés pour le --redeploy.
wcs/ctl/management/commands/hobo_deploy.py
@ -105,6 +105,7 @@ class Command(TenantCommand):
return
service['this'] = True
stored_hobo_json_content = json.dumps(self.all_services, indent=2)
fpeters commented 2023-11-11 17:23:19 +01:00
Author
Owner
Copy Link

L'ajout de l'attribut this est conservé, pour faire comme les autres modules.

L'ajout de l'attribut this est conservé, pour faire comme les autres modules.
wcs/ctl/management/commands/hobo_deploy.py
@ -175,3 +176,1 @@
atomic_write(
os.path.join(pub.tenant.directory, 'hobo.json'), force_bytes(json.dumps(self.all_services))
)
atomic_write(os.path.join(pub.tenant.directory, 'hobo.json'), force_bytes(stored_hobo_json_content))
fpeters commented 2023-11-11 17:24:08 +01:00
Author
Owner
Copy Link

C'est la version sérialisée plus haut qu'on enregistre, sans avoir à se préoccuper des modifications au contenu qui peuvent avoir eu lieu.

C'est la version sérialisée plus haut qu'on enregistre, sans avoir à se préoccuper des modifications au contenu qui peuvent avoir eu lieu.
fpeters changed title from WIP: misc: do not alter base_url in stored hobo.json (#21605) to misc: do not alter base_url in stored hobo.json (#21605) 2023-11-11 17:24:24 +01:00
nroche approved these changes 2023-11-24 16:12:04 +01:00
fpeters merged commit 2c4d065b4c into main 2023-11-24 16:15:23 +01:00
fpeters deleted branch wip/21605-keep-hobo-json-as-is 2023-11-24 16:15:23 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
nroche
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#832
No description provided.
Delete Branch "wip/21605-keep-hobo-json-as-is"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?