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

jenkins stage doesn't fail on error in ingest-dataset #427

Open
peggynewman opened this issue Jun 18, 2021 · 3 comments
Open

jenkins stage doesn't fail on error in ingest-dataset #427

peggynewman opened this issue Jun 18, 2021 · 3 comments
Labels
enhancement New feature or request pipelines Issue related to pipelines

Comments

@peggynewman
Copy link

An ERROR in the logs on the UUID and the Index stages didn't fail the stage and the job kept going.

Ingest dataset job: http://aws-spark-quoll-master.ala:9194/job/Ingest-dataset/87/console

UUID stage: http://aws-spark-quoll-master.ala:9194/job/UUID-dataset/605/console

Found this looking for a loaded record count.

@javier-molina
Copy link

@djtfmartin can I leave this with you?

@djtfmartin
Copy link
Member

djtfmartin commented Jun 18, 2021

This is expected behaviour - but obviously we can change if there's a better solution. There maybe something we can do on the jenkins side.

The way it works is the Indexing pipeline validates that other pipelines have completed successfully for a dataset.
it checks

  • Interpretation has been ran dataset
  • UUIDs have been run and have been run after interpretation (so we are in-sync)
  • SDS has been run after interpretation

If all these validations pass then we generate the IndexRecords for a dataset ready for SOLR indexing. If any fail, we don't and the dataset does make it into the SOLR index.

We've done it this way so that we avoid the situation that a single problem dataset breaks our processes for all datasets e.g. "Uuid-all". We can regenerate all UUIDs and skip over the dataset that's causing an issue.

We could change the end of each pipeline so that we throw an exception if it fails. Whether we throw the exception could be determined by config and hence could be a commandline/jenkins parameter for the job.

@peggynewman
Copy link
Author

Hi Dave, after discussing with @patkyn this sounds like it would be very useful to use config to set whether the UUID job fails, but that's not high priority.

@javier-molina javier-molina added enhancement New feature or request and removed bug Something isn't working labels Jul 21, 2021
@javier-molina javier-molina added pipelines Issue related to pipelines and removed testing-findings labels Sep 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request pipelines Issue related to pipelines
Projects
None yet
Development

No branches or pull requests

3 participants