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

AII schema should not be extensible #201

Open
jouvin opened this issue Aug 25, 2016 · 4 comments
Open

AII schema should not be extensible #201

jouvin opened this issue Aug 25, 2016 · 4 comments
Assignees

Comments

@jouvin
Copy link
Contributor

jouvin commented Aug 25, 2016

Up to Quattor 16.6.0, AII schema (/system/aii/xxx) is making an extensive use of extensible. Basically all parts of the schema are extensible and only some parts are bounded to a real schema. As a result, every typo or mistake in configuration paths cannot be catched during compilation and result in aii-shellfe errors not always easy to understand (see #200 for example)...

The motivation for this is probably to allow an easy addition of hooks in various parts of AII. It would be good to find a way to provide the flexibility of hooks without loosing the ability to validate the configuration.

@jouvin jouvin added this to the 16.10 milestone Aug 25, 2016
@jrha jrha self-assigned this Oct 4, 2016
@jrha
Copy link
Member

jrha commented Oct 4, 2016

Discussed at workshop and agreed to survey the community for hook usage and remove the extensible option.

@jrha
Copy link
Member

jrha commented Feb 16, 2017

Found more hooks!, See #245.

@jrha
Copy link
Member

jrha commented Mar 21, 2017

Discussed at March 2017 workshop, agreed that:

  1. structure_aii should not be extensible.
  2. structure_aii_hooklist will need to remain extensible unless it can be restructured in future.
  3. structure_aii_module shoud be removed and proper types defined for each module.

@jrha jrha removed this from the 17.3 milestone Mar 21, 2017
@jrha
Copy link
Member

jrha commented Nov 12, 2024

New implementation of aii_commands_hook happening in #349.

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

No branches or pull requests

3 participants