search

Metro-Records / la-metro-councilmatic

:metro: An instance of councilmatic for LA Metro
MIT License
6 stars 2 forks source link

check the status of requests to the pdf merger #382

Closed hancush closed 2 years ago

hancush commented 5 years ago

the pdf merger went down unexpectedly, but the logs on the councilmatic side did not indicate anything had gone wrong. there is an opportunity to verify that post requests return a 200 status code in the compile_pdfs command. consider using that opportunity to raise exceptions, so we know something isn't working.

reginafcompton commented 5 years ago

Yes, @hancush - that seems like a smart place for exception handling!

Could we also have logs further upstream? BGA has a pattern for nginx logging – could we do something similar with supervisor?

hancush commented 5 years ago

i think that's a great idea, hinted at but def not explicitly arrived at, at the bottom of https://github.com/datamade/metro-pdf-merger/issues/19.

shrayshray commented 5 years ago

As of 5:10pm PST 11/30, the packet for this Agenda was available: https://boardagendas.metro.net/event/regular-board-meeting-52018b89a7ba/ However, today the packet link is missing. Only one change on our side in the meantime ... shortly after 5:10pm 11/30, Omar had to export and re-publish the Agenda. Not sure whether this would have an impact, just wanted to give you all the info we have! Can you please look into getting the packet back up?

reginafcompton commented 5 years ago

@shrayshray - the packet is visible now. I was doing some testing with the metro-PDF-merger, which likely caused the delay.

shrayshray commented 5 years ago

@reginafcompton phew, thank you!!