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

sssd: adjust permissions on functional tests & tidyup #18126

Merged
merged 1 commit into from Dec 6, 2023

Conversation

josegomezr
Copy link
Contributor

@josegomezr josegomezr commented Nov 6, 2023

  • Reorganized the logic so all artifacts needed are downloaded on start.
  • Copying with permissions is now done with 'install'.
  • DRY-up docker/podman run command.

Addresses: poo#138989

@josegomezr josegomezr force-pushed the sssd_perms_refactorish branch 3 times, most recently from 63aca83 to 01828b5 Compare November 6, 2023 15:58
@DimStar77
Copy link
Contributor

* Verification run: [TBD]

… Still TBD?

@josegomezr
Copy link
Contributor Author

josegomezr commented Nov 13, 2023

… Still TBD?

Just updated it to:
https://openqa.opensuse.org/tests/3717317
https://openqa.opensuse.org/tests/3719337

I had a couple of failed runs on silly errors before.

- Reorganized the logic so all artifacts needed are downloaded
  on start.
- Copying with permissions is now done with 'install'.
- DRY-up docker/podman run command.

Addresses: poo#138989
@josegomezr
Copy link
Contributor Author

Finally it should be fixed now. VR: https://openqa.opensuse.org/tests/3735258

@Vogtinator Vogtinator merged commit 9e4eac8 into os-autoinst:master Dec 6, 2023
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants