Skip to content

ci: add a daily test for comparing against latest masterfile#17

Open
Fabio1988 wants to merge 1 commit intomasterfrom
ci-test
Open

ci: add a daily test for comparing against latest masterfile#17
Fabio1988 wants to merge 1 commit intomasterfrom
ci-test

Conversation

@Fabio1988
Copy link

@Fabio1988 Fabio1988 commented Dec 14, 2025

@lenisko the build is failing on purpose... as the latest masterfile is corrupted... I will try to fix this in pogo-data-generator .. But this daily fetching and test execution will help us identifying such an issue earlier next time :)

@Fabio1988 Fabio1988 requested a review from lenisko December 14, 2025 17:46
@lenisko
Copy link
Collaborator

lenisko commented Dec 14, 2025

cc @Mygod can I push that on you? I'm on 4 projects rn, need to process as many as possible till my personal deadline.

@Fabio1988
Copy link
Author

Fabio1988 commented Dec 14, 2025

cc @Mygod can I push that on you? I'm on 4 projects rn, need to process as many as possible till my personal deadline.

I am on to fixing the masterfile generator, already created a PR.. this is only a gitbub action which runs daily, we can easily merge it :)

It exexutes the test suite with latest masterfile

@Mygod
Copy link
Collaborator

Mygod commented Dec 15, 2025

Please rerun the test?

@lenisko
Copy link
Collaborator

lenisko commented Dec 16, 2025

done, failed

@Mygod
Copy link
Collaborator

Mygod commented Dec 16, 2025

so no merge

@Fabio1988
Copy link
Author

so no merge

Test is failing as the test is running with the wrong masterfile on the build pipeline :)
The cronned action is not running yet... :)

The action is not commiting the latest masterfile fetched, don't think that will be needed

@Fabio1988
Copy link
Author

tests are now passing again :)

this new daily scheduled workflow should only visualize if we run into such a bad scenario again at some point in the future, so we get an early notice :)

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.

3 participants