Skip to content

Comments

Gt1180 apel loading validator#82

Open
rowan04 wants to merge 3 commits intoapel:devfrom
rowan04:GT1180-apel-loading-validator
Open

Gt1180 apel loading validator#82
rowan04 wants to merge 3 commits intoapel:devfrom
rowan04:GT1180-apel-loading-validator

Conversation

@rowan04
Copy link
Contributor

@rowan04 rowan04 commented Feb 12, 2026

this currently contains all commits from #75 as well, but I believe it should sort itself out when that PR gets merged into dev?

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

Resolves GT-1228
Resolves GT-1328

}

except (configparser.NoSectionError) as err:
print("Error in configuration file. Check that file exists first: %s" % err)

Check notice

Code scanning / CodeQL

Use of a print statement at module level Note

Print statement may execute during import.
Comment on lines +46 to +48
<input type="radio" id="submission_type" name="submission_type" value="validate"> Validate
<br>
<input type="radio" id="submission_type" name="submission_type" value="load"> Load
Copy link
Contributor

Choose a reason for hiding this comment

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

HTML requires unique IDs
<input type="radio" id="submission_type_validate" name="submission_type" value="validate"> Validate <br> <input type="radio" id="submission_type_load" name="submission_type" value="load"> Load

Copy link
Member

Choose a reason for hiding this comment

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

Does it need both id and name?

Copy link
Member

Choose a reason for hiding this comment

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

Ah, I see, id does need to be unique. But then the actual question is, do we need everything tagged with an id? Are we referring to elements by id anywhere?

@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 from 49c2787 to ba20fb0 Compare February 19, 2026 13:37
- 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
@tofu-rocketry tofu-rocketry force-pushed the GT1180-apel-loading-validator branch 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 blockhole 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.

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