Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Weekly deployment failures due to non-up-to-date dtds #3583

Closed
jfbischoff opened this issue Nov 26, 2024 · 4 comments
Closed

Weekly deployment failures due to non-up-to-date dtds #3583

jfbischoff opened this issue Nov 26, 2024 · 4 comments
Labels

Comments

@jfbischoff
Copy link
Collaborator

Hi
I have received an error message that the weekly deployments have not been running for 2 weeks:
https://github.com/matsim-org/matsim-libs/actions/workflows/deploy-weekly.yaml

The reason for this is a test failure in
org.matsim.contrib.ev.infrastructure.ChargerReaderWriterTest.testReadWriteChargersWithAttributes()

This failure occurs because the chargers_v1.dtd in http://matsim.org/files/dtd/chargers_v1.dtd is not up to date, it has also been changed recently:

<!ELEMENT charger (attributes?)>
by @sebhoerl

I'm not sure how the dtds make it to the website, but apparently that pipe is broken.
@mrieser - you probably have an idea?

Bests,
Joschka

@jfbischoff jfbischoff added the bug label Nov 26, 2024
@kt86
Copy link
Contributor

kt86 commented Nov 26, 2024

As far as I see, the workflow for pushing the dtds to the website has never run: https://github.com/matsim-org/matsim-libs/blob/master/.github/workflows/deploy-dtds.yaml
grafik

@kt86
Copy link
Contributor

kt86 commented Nov 26, 2024

@jfbischoff Thanks for diving deeper in and finding the cause for the missing weekly build.
I tried to update the script, that is responsible for updating the dtds on the website -> see #3585.

@kt86
Copy link
Contributor

kt86 commented Nov 27, 2024

The workflow for publishing the DTDs is now fixed #3597
I manually (re-)started the job for creating the weekly build. Let's see, what happens there

@jfbischoff
Copy link
Collaborator Author

Up and running again, thanks @kt86 !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants