ucphhpc / ansible-docker-migrid

A collection of tools for automating docker-migrid deployment on a physical or virtual machine with Ansible
GNU General Public License v2.0
0 stars 1 forks source link

Role update_status_html writing to the wrong file #37

Open benibr opened 5 months ago

benibr commented 5 months ago

The role update_status_html writes to {{ migrid_state_directory }}/wwwpublic/status.html as can be seen in https://github.com/ucphhpc/ansible-docker-migrid/blob/main/roles/update_status_html/tasks/main.yml#L145

This is wrong as this file is part of the HTML content showing the status of Migrid eg. https://status.erda.dk/status.html

I'm not sure what the initial idea of this role was, and I guess it makes sense to write all the deploy information to a file for error reporting and reproducability but we should use another file as status.html is meant as a webpage :-)

benibr commented 5 months ago

@Bjarke42 Can you comment on this what the initial story behind that role is?

Bjarke42 commented 5 months ago

We choose this file because it has always been empty here, so i decided to use that file for deploy info. I guess it would be easy to rename it to deploy.html. But with all the errors we have had, we have never seen any info come up in this file, so do you have an exampe of something going there ?

benibr commented 5 months ago

As explained above this is the file that contains the HTML for showing the status information. See example link in the description.

Bjarke42 commented 5 months ago

Okay, thats manually inserted information by KU. We do not use that because we have a very different way of informing, via https://serviceinfo.au.dk/ I just wondered if it had any autofilled information in the file.