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

fix: [PAYMCLOUD-109] poc GitHub runner on dev ops lab #362

Merged
merged 8 commits into from
Oct 15, 2024

Conversation

ffppa
Copy link
Contributor

@ffppa ffppa commented Oct 14, 2024

List of changes

  • Add secretRef on GITHUB_PAT env var

Motivation and context

Fix the GITHUB_PAT env management, to make it works a secretRef is needed.

Type of changes

  • Add new module
  • Update existing module
  • Remove existing module

Does this introduce a breaking change?

  • Yes
  • No

Other information

Run checks

Useful commands to run checks on local machine

bash .utils/terraform_run_all.sh init local
pre-commit run -a

ffppa added 7 commits October 14, 2024 16:49
This commit integrates the retrieval of GitHub Personal Access Token (PAT) from Azure Key Vault in the Terraform configuration. It removes the hardcoded PAT from the environment variables and injects it securely from the Key Vault secret. This enhances security by avoiding hardcoding sensitive information in the codebase.
Introduced GitHub PAT in the container environment variables to securely access repositories. Updated `main.tf` to correctly reference the existing Key Vault secret for the GitHub PAT. Removed redundant secret configuration for cleaner code.
Moved GitHub PAT from plain text in locals.tf to secure secret management in main.tf. This change improves security by using the secret_name attribute for handling sensitive credentials.
Updated the Markdown table in the README to use consistent line breaks with `<br/>`. This change ensures better readability and maintainability of the table entries.
Replaced all instances of "/>" with ">" in code blocks for consistency. This change improves readability and maintains a uniform coding style across the documentation.
…on-DevOpsLab' into PAYMCLOUD-109-POC-Github-Runner-on-DevOpsLab
@ffppa ffppa marked this pull request as ready for review October 14, 2024 15:34
@ffppa ffppa requested review from a team as code owners October 14, 2024 15:34
Adjusted spacing to align variables in `variables.tf` and `main.tf` for better readability. Reformatted the README.md input table to ensure consistency and clarity.
Copy link
Contributor

@mamari90 mamari90 left a comment

Choose a reason for hiding this comment

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

👍

@ffppa ffppa merged commit e557087 into main Oct 15, 2024
4 checks passed
@ffppa ffppa deleted the PAYMCLOUD-109-POC-Github-Runner-on-DevOpsLab branch October 15, 2024 09:55
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.

2 participants