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

lava: Update FVP commands for latest shrinkwrap release #2737

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

broonie
Copy link
Member

@broonie broonie commented Nov 7, 2024

The latest shrinkwrap release has updated the containers we're using for
the FVP to a new version which moves SVE support from a plugin into the
main model. This has been causing all our FVP jobs to fail as the
arguments supplied are invalid, update with parameters suitable for the
new version of the model.

Signed-off-by: Mark Brown [email protected]

The latest shrinkwrap release has updated the containers we're using for
the FVP to a new version which moves SVE support from a plugin into the
main model.  This has been causing all our FVP jobs to fail as the
arguments supplied are invalid, update with parameters suitable for the
new version of the model.

Signed-off-by: Mark Brown <[email protected]>
Copy link
Member

@nuclearcat nuclearcat left a comment

Choose a reason for hiding this comment

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

LGTM, but is it legacy? I am not rolling legacy updates anymore. I can take risk and try to do that, but it might break whatever remaining of it.

@broonie
Copy link
Member Author

broonie commented Nov 8, 2024

There seems to be an awful lot of stuff still running on legacy, I'm seeing far more jobs going through from that than I am from Maestro. I'm certainly not seeing any FVP jobs, looks like that needs some updates in config/runtime/boot which didn't happen yet...

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

Successfully merging this pull request may close these issues.

2 participants