Skip to content

update lock file - #179

Merged
ocefpaf merged 6 commits into
ioos:mainfrom
ocefpaf:update_lockfile
Feb 19, 2024
Merged

update lock file#179
ocefpaf merged 6 commits into
ioos:mainfrom
ocefpaf:update_lockfile

Conversation

@ocefpaf

@ocefpaf ocefpaf commented Feb 19, 2024

Copy link
Copy Markdown
Member

I also want to investigate some of the CI failures here.

@ocefpaf

ocefpaf commented Feb 19, 2024

Copy link
Copy Markdown
Member Author

2016-11-16-CF-UGRID-SGRID-conventions.ipynb is failing b/c the https://geoport.whoi.edu/thredds/catalog/coawst_4/use/fmrc/catalog.html?dataset=coawst_4/use/fmrc/coawst_4_use_best.ncd endpoint seems to be down. I'm not sure if it moved, is down permanently, or if it is a temporary issue. I'm looking into other datasets that could be used here instead.

@ocefpaf

ocefpaf commented Feb 19, 2024

Copy link
Copy Markdown
Member Author

jupyterbook/content/code_gallery/data_access_notebooks/2017-06-12-NCEI_RA_archive_history.ipynb failed with HTTPError: HTTP Error 503: Service Unavailable. This is most likely temporary, merging this one so we can move forward with other PRs.

@ocefpaf
ocefpaf merged commit 5cd4a49 into ioos:main Feb 19, 2024
@ocefpaf
ocefpaf deleted the update_lockfile branch February 19, 2024 17:36
@MathewBiddle

Copy link
Copy Markdown
Contributor

2016-11-16-CF-UGRID-SGRID-conventions.ipynb is failing b/c the https://geoport.whoi.edu/thredds/catalog/coawst_4/use/fmrc/catalog.html?dataset=coawst_4/use/fmrc/coawst_4_use_best.ncd endpoint seems to be down. I'm not sure if it moved, is down permanently, or if it is a temporary issue. I'm looking into other datasets that could be used here instead.

We seem to run into this a lot with these notebooks. Would it be worthwhile to write a test Action that tests all the endpoints we hit to ensure they are functioning properly? Then, run that before the CI's?

It's difficult to highlight IOOS data services and tools when some of the services are down...

@ocefpaf

ocefpaf commented Feb 20, 2024

Copy link
Copy Markdown
Member Author

We seem to run into this a lot with these notebooks.

In a way, the notebook itself is a test for the endpoint. Separating the endpoint test may hinder the failure when it happens during download time or accessing a variable.

It's difficult to highlight IOOS data services and tools when some of the services are down...

Indeed. We should however have a list of essential/important services that we could track in another repo and have some diagnostics for them.

@MathewBiddle

Copy link
Copy Markdown
Contributor

Great points!

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