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.

Sign up for GitHub

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

Jump to bottom

Make the pidfile accessible by everyone #3252

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Make the pidfile accessible by everyone #3252

wants to merge 7 commits into from

Commits on Nov 21, 2024

  1. Makes the pidfile accessible by everyone 

    Creates the pidfile at an accessible location
for every user, this way the manage pidfile
warning is avoided.

Signed-off-by: mcasquer <[email protected]>
    @mcasquer
    mcasquer committed Nov 21, 2024
    Configuration menu
    Copy the full SHA
    4786897 View commit details
    Browse the repository at this point in the history

  2. Adds a check to only create the pid directory 

    when it doesn't exist.

Signed-off-by: mcasquer <[email protected]>
    @mcasquer
    mcasquer committed Nov 21, 2024
    Configuration menu
    Copy the full SHA
    2dd6f74 View commit details
    Browse the repository at this point in the history

  3. Fixes the tmt reboot test, updating the pidfile location in the scripts. 

    Also formats the pid directory creation command.

Signed-off-by: mcasquer <[email protected]>
    @mcasquer
    mcasquer committed Nov 21, 2024
    Configuration menu
    Copy the full SHA
    d2ee700 View commit details
    Browse the repository at this point in the history

  4. Updates the environment variables in out-of-session test. 

    This way we avoid TestingFarm to set the old values.

Signed-off-by: mcasquer <[email protected]>
    @mcasquer
    mcasquer committed Nov 21, 2024
    Configuration menu
    Copy the full SHA
    74ec495 View commit details
    Browse the repository at this point in the history

  5. Removes the sudo(s) from tmt commands and instead apply it directly 

    in the pidfile creation. Also includes the super().setup() call to
the setup GuestSsh function.

Signed-off-by: mcasquer <[email protected]>
    @mcasquer
    mcasquer committed Nov 21, 2024
    Configuration menu
    Copy the full SHA
    f9c23a0 View commit details
    Browse the repository at this point in the history

  6. Removes the self.become this way the condition is applied 

    every time the user has no privileges

Signed-off-by: mcasquer <[email protected]>
    @mcasquer
    mcasquer committed Nov 21, 2024
    Configuration menu
    Copy the full SHA
    026d3ed View commit details
    Browse the repository at this point in the history

  7. Changes bfu_* options to use unprivileged Fedora container image 

    and fedora user

Signed-off-by: mcasquer <[email protected]>
    @mcasquer
    mcasquer committed Nov 21, 2024
    Configuration menu
    Copy the full SHA
    0f1a806 View commit details
    Browse the repository at this point in the history