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

Make arch field unsupported in the spec #2552

Merged
merged 1 commit into from
Dec 7, 2023
Merged

Make arch field unsupported in the spec #2552

merged 1 commit into from
Dec 7, 2023

Conversation

thrix
Copy link
Collaborator

@thrix thrix commented Dec 6, 2023

The arch field is currently special, and unsupported if it is set in the hardware field. Fix this discrepancy and make sure a warning is show so users are aware to not use it here.

Pull Request Checklist

  • write the documentation
  • update the specification

@thrix thrix added this to the 1.30 milestone Dec 6, 2023
Copy link
Collaborator

@psss psss 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, tests need some adjustments as well though.

@psss psss added specification Metadata specification (core, tests, plans, stories) area | hardware Implementation of hardware requirements labels Dec 6, 2023
@psss
Copy link
Collaborator

psss commented Dec 7, 2023

Looks good, tests need some adjustments as well though.

b7b96ca should fix it.

@psss psss self-assigned this Dec 7, 2023
@psss psss changed the title Make arch field unsupported in the spec. Make arch field unsupported in the spec Dec 7, 2023
The `arch` field is currently special, and unsupported if it is
set in the hardware field. Fix this discrepancy and make sure a
warning is show so users are aware to not use it here.

Signed-off-by: Miroslav Vadkerti <mvadkert@redhat.com>
@psss
Copy link
Collaborator

psss commented Dec 7, 2023

/packit test --identifier full

@psss
Copy link
Collaborator

psss commented Dec 7, 2023

/packit test

@psss
Copy link
Collaborator

psss commented Dec 7, 2023

/packit build

@psss
Copy link
Collaborator

psss commented Dec 7, 2023

/packit test --identifier full

@psss psss merged commit 395aa33 into main Dec 7, 2023
13 of 23 checks passed
@psss psss deleted the arch-discrepancy branch December 7, 2023 10:00
@psss psss linked an issue Jan 9, 2024 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area | hardware Implementation of hardware requirements specification Metadata specification (core, tests, plans, stories)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Discrepancy with arch in hardware requirements and plugin options
2 participants