Skip to content

Gt1180 apel loading validator#82

Merged
tofu-rocketry merged 5 commits intoapel:devfrom
rowan04:GT1180-apel-loading-validator
Feb 25, 2026
Merged

Gt1180 apel loading validator#82
tofu-rocketry merged 5 commits intoapel:devfrom
rowan04:GT1180-apel-loading-validator

Conversation

@rowan04
Copy link
Contributor

@rowan04 rowan04 commented Feb 12, 2026

host where this is running: http://host-172-16-103-131.nubes.stfc.ac.uk/validator/

Resolves GT-1228
Resolves GT-1328

@tofu-rocketry tofu-rocketry self-assigned this Feb 17, 2026
@tofu-rocketry tofu-rocketry added this to the v1.7 milestone Feb 17, 2026
@tofu-rocketry tofu-rocketry force-pushed the GT1180-apel-loading-validator branch 2 times, most recently from ba20fb0 to 1c9df0e Compare February 19, 2026 13:42
Copy link
Member

@tofu-rocketry tofu-rocketry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could do with some documentation in monitoring/docs that explains how to set up the database (and blackhole engine) for this manually.

See also discussion started by @garaimanoj

@tofu-rocketry
Copy link
Member

One more point - we can't have apel in the requirements file as it's not available in PyPI - we'll manage the install manually/through Aquilon.

@rowan04
Copy link
Contributor Author

rowan04 commented Feb 25, 2026

i am fixing/rebasing

- has radio buttons to specifiy whether record should be validated or
  loaded
- adds new load method
- creates instance of apel loader
- makes use of a blackhole mariadb database, which doesn't store data
- also fixes validator bug which would prevent v0.4 records from getting
  a formatted output
- removes unnecessary and duplicated html id attributes
- removes apel from requirements.txt
@rowan04 rowan04 force-pushed the GT1180-apel-loading-validator branch from 240e67a to d6c2668 Compare February 25, 2026 13:24
Copy link
Member

@tofu-rocketry tofu-rocketry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Thanks. 🙂

@tofu-rocketry tofu-rocketry merged commit 26f586b into apel:dev Feb 25, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants