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

ensure that /usr/local/pf/conf/system_init_key is created #8571

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

Conversation

jrouzierinverse
Copy link
Member

Description

Ensure that /usr/local/pf/conf/system_init_key is created.

Issue

Fixes #8565

Delete branch after merge

YES

Checklist

(REQUIRED) - [yes, no or n/a]

  • Document the feature
  • Add OpenAPI specification
  • Add unit tests
  • Add acceptance tests (TestLink)

NEWS file entries

Bug Fixes

Copy link
Contributor

@JeGoi JeGoi left a comment

Choose a reason for hiding this comment

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

Just one comment
It looks fine for me just: wait until the test pipeline is finished before merging.
https://gitlab.com/inverse-inc/packetfence/-/pipelines/1690519251
Thanks

hexdump -e '/1 "%x"' < /dev/urandom | head -c 32 > /usr/local/pf/conf/system_init_key
fi

setup_system_init_key
Copy link
Contributor

Choose a reason for hiding this comment

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

So, this is the earliest place to put in, correct?
I recall that you said it needs to be as soon as possible.

@JeGoi
Copy link
Contributor

JeGoi commented Feb 27, 2025

@jrouzierinverse the pipeline is failing for debian12.

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

Successfully merging this pull request may close these issues.

Unable to install PacketFence 14.1 on Debian 12.9
2 participants