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

OSIDB-3712: Create container image #1

Merged
merged 6 commits into from
Nov 25, 2024
Merged

OSIDB-3712: Create container image #1

merged 6 commits into from
Nov 25, 2024

Conversation

MrMarble
Copy link
Member

@MrMarble MrMarble commented Nov 21, 2024

This pull request introduces several changes to improve the Docker-based CI/CD pipeline for running tests, including updates to the Docker configuration, scripts, and test settings.

Docker Configuration and Scripts:

  • Added a .dockerignore file to exclude unnecessary files from the Docker build context.
  • Created a Dockerfile to set up the environment for running tests, including installing dependencies and configuring Kerberos authentication.
  • Added auth.sh script for Kerberos authentication within the container.
  • Added install-certs.sh script to install Red Hat certificates if the RH_CERT_URL environment variable is set.
  • Added krb5.conf configuration file for Kerberos.
  • Added .gitignore to the krb5.conf.d directory.

Documentation:

  • Created a README.md file in the docker directory to provide instructions on building and running the Docker container, as well as running tests within the container.

Test Configuration:

  • Updated package.json to simplify test scripts and remove unnecessary dependencies. [1] [2]
  • Modified playwright.config.ts to use different reporters based on the CI environment.
  • Adjusted test timeouts and expectations in tests/flaw.spec.ts to improve reliability. [1] [2]

Closes OSIDB-3712

@MrMarble MrMarble self-assigned this Nov 21, 2024
@MrMarble MrMarble marked this pull request as ready for review November 21, 2024 14:03
Copy link

@costaconrado costaconrado left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Member

@C-Valen C-Valen left a comment

Choose a reason for hiding this comment

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

Looks good

@MrMarble MrMarble force-pushed the OSIDB-3712 branch 2 times, most recently from afbec58 to 450c81f Compare November 22, 2024 16:29
@MrMarble MrMarble merged commit f7518c8 into main Nov 25, 2024
2 checks passed
@MrMarble MrMarble deleted the OSIDB-3712 branch November 25, 2024 08:14
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