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

Generated prescription do not match the expected schema #205

Closed
codificat opened this issue Jan 10, 2023 · 4 comments · Fixed by #207
Closed

Generated prescription do not match the expected schema #205

codificat opened this issue Jan 10, 2023 · 4 comments · Fixed by #207
Labels
kind/bug Categorizes issue or PR as related to a bug. needs-triage Indicates an issue or PR lacks a `triage/...` label and requires one. sig/stack-guidance Categorizes an issue or PR as relevant to SIG Stack Guidance.

Comments

@codificat
Copy link
Member

Describe the bug

Currently, this job is generating prescriptions that do not match the schema

To Reproduce

See PRs generated by recent executions of this job, for example https://github.com/thoth-station/prescriptions/pull/38943/files

Expected behavior

Prescriptions generated by this job should not fail validation

Screenshots

The prescription file referenced above:

units:
  wraps:
  - name: PydeconzGitHubURLWrap
    type: wrap
    should_include:
      adviser_pipeline: true
    match:
      state:
        resolved_dependencies:
        - name: pydeconz
    run:
      justification:
      - type: INFO
        message: Package 'pydeconz' is hosted on GitHub
        link: https://github.com/Kane610/deconz
        package_name: pydeconz
        metadata:
        - prescriptions_repository: [email protected]:thoth-station/prescriptions.git
          prescriptions_version: 0.8.0

Note that the metadata is not in the location expected by the schema.

Additional context

Prescription metadata was introduced in #155.

Currently, the prescriptions in the prescription repo do not match validation. thoth-adviser is stuck at using v0.22.0. See thoth-station/adviser#2402

@codificat codificat added kind/bug Categorizes issue or PR as related to a bug. needs-triage Indicates an issue or PR lacks a `triage/...` label and requires one. labels Jan 10, 2023
@VannTen
Copy link
Member

VannTen commented Jan 18, 2023

Do we have other errors than just the metadata being in the wrong place ?

@codificat
Copy link
Member Author

Do we have other errors than just the metadata being in the wrong place ?

Not that I'm aware of. But I have only checked a very limited number of prescriptions.

One problem/challenge here is that, as far as I can tell, thoth-adviser validate-prescriptions errors out on the first validation failure and stops. If there was a way to make it show all the errors, we would be sure

@VannTen
Copy link
Member

VannTen commented Jan 24, 2023 via email

@VannTen
Copy link
Member

VannTen commented Jan 24, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. needs-triage Indicates an issue or PR lacks a `triage/...` label and requires one. sig/stack-guidance Categorizes an issue or PR as relevant to SIG Stack Guidance.
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants