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

Enable LUKS2 and PBKDF2 by default for compat with grub (boo#1185291) #285

Merged
merged 1 commit into from
May 23, 2024

Conversation

lnussel
Copy link
Contributor

@lnussel lnussel commented May 16, 2024

No description provided.

@lnussel
Copy link
Contributor Author

lnussel commented May 22, 2024

is the ci using an outdated yast2-installation-control? Works for me locally

@ancorgs
Copy link
Contributor

ancorgs commented May 22, 2024

I'm not saying that will fix the problem with CI, but it would be probably correct to update the version of the BuildRequires for yast2-installation-control.

@ancorgs
Copy link
Contributor

ancorgs commented May 22, 2024

is the ci using an outdated yast2-installation-control? Works for me locally

CI uses a container that, according to registry.opensuse.org, has not been rebuilt since 11. Apr 2024. Causes are under investigation.

@lnussel
Copy link
Contributor Author

lnussel commented May 22, 2024

right, at least it will make the problem obvious. updated.

@lnussel lnussel closed this May 22, 2024
@lnussel lnussel reopened this May 22, 2024
@lnussel
Copy link
Contributor Author

lnussel commented May 22, 2024

haha, it's not building a package o_O

@lnussel
Copy link
Contributor Author

lnussel commented May 22, 2024

Don't know what this yast-ci-ruby does but building a package right there from git works like this:

rpmbuild --build-in-place --noprep --define "_sourcedir $PWD" --define "_topdir $PWD/rpm" --define "_tmppath %_topdir/tmp" -bb package/skelcd-control-openSUSE.spec

@lslezak
Copy link
Member

lslezak commented May 22, 2024

I have fixed building the container images in OBS and restarted the CI, now it is green. ✅

Copy link
Contributor

@ancorgs ancorgs left a comment

Choose a reason for hiding this comment

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

LGTM

@ancorgs ancorgs merged commit 5d07096 into yast:master May 23, 2024
2 checks passed
@yast-bot
Copy link
Contributor

✔️ Internal Jenkins job #6 successfully finished
✔️ Created OBS submit request #1176161

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.

4 participants