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

[DDS-1993] Added ssh-client for lagoon token fetch. #272

Merged
merged 3 commits into from
Aug 9, 2024

Conversation

GROwen
Copy link
Contributor

@GROwen GROwen commented Aug 8, 2024

Checklist

  • Affected Issues have been mentioned in the Closing issues section
  • Documentation has been written/updated.
  • Changelog entry has been written

Explain the details for making this change. What existing problem does the pull request solve?

Changelog Entry

Adds ssh-client for Lagoon token fetch.

Closing issues

Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if such).

@GROwen GROwen self-assigned this Aug 8, 2024
@@ -1,6 +1,6 @@
ARG PHP_VERSION=8.3
FROM php:${PHP_VERSION}-cli-alpine AS php-cli
FROM ghcr.io/skpr/mtk:latest AS mtk
FROM ghcr.io/skpr/mtk:v2.0.2 AS mtk
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@nicksantamaria The latest tag has been dropped from the registry. This is the latest "stable" tag, there is a more recent tag but it looks like it's related to a PR

Copy link
Contributor

Choose a reason for hiding this comment

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

I'm fine with this 👍

@GROwen GROwen requested review from nicksantamaria and klaukl August 9, 2024 02:42
@GROwen GROwen added the NEEDS REVIEW Good for newcomers label Aug 9, 2024
@GROwen GROwen marked this pull request as ready for review August 9, 2024 02:43
@@ -1,6 +1,6 @@
ARG PHP_VERSION=8.3
FROM php:${PHP_VERSION}-cli-alpine AS php-cli
FROM ghcr.io/skpr/mtk:latest AS mtk
FROM ghcr.io/skpr/mtk:v2.0.2 AS mtk
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm fine with this 👍

@GROwen GROwen merged commit f4f941f into 6.x Aug 9, 2024
11 checks passed
@GROwen GROwen deleted the build/DDS-1993--add-encrypted-machine-user-key branch August 9, 2024 03:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NEEDS REVIEW Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants