Skip to content

refactor: update alpine example#471

Merged
lbajolet-hashicorp merged 3 commits intovmware:mainfrom
brenthc:alpine-example-update
Feb 20, 2025
Merged

refactor: update alpine example#471
lbajolet-hashicorp merged 3 commits intovmware:mainfrom
brenthc:alpine-example-update

Conversation

@brenthc
Copy link
Copy Markdown
Contributor

@brenthc brenthc commented Oct 2, 2024

This update has been tested to successfully build Alpine 3.20.3 and execute the included ls command over ssh.

@brenthc brenthc requested a review from a team as a code owner October 2, 2024 15:22
Copy link
Copy Markdown
Collaborator

@tenthirtyam tenthirtyam left a comment

Choose a reason for hiding this comment

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

Minor suggested for readability and continued maintenance.

@tenthirtyam tenthirtyam changed the title update alpine example to 3.20.3 fix: update alpine example Oct 2, 2024
@tenthirtyam tenthirtyam added the bug Bug label Oct 2, 2024
@tenthirtyam tenthirtyam added this to the v1.5.0 milestone Oct 2, 2024
@tenthirtyam tenthirtyam marked this pull request as draft October 3, 2024 14:02
@tenthirtyam
Copy link
Copy Markdown
Collaborator

Hi Brent! Wanted to check in and see if you are planning to update this PR.

@brenthc
Copy link
Copy Markdown
Contributor Author

brenthc commented Nov 4, 2024

Yes, I'm still planning to see it through. My initial version was a straight update to the existing example. If you'd prefer a refactor (as discussed), I'll need some more time to complete that. I tore down my lab a little while ago 😬

@tenthirtyam tenthirtyam modified the milestones: v1.5.0, Backlog Dec 9, 2024
brenthc and others added 3 commits February 3, 2025 16:11
Co-authored-by: Ryan Johnson <ryan@tenthirtyam.org>
Signed-off-by: Ryan Johnson <ryan.johnson@broadcom.com>
@tenthirtyam tenthirtyam force-pushed the alpine-example-update branch from 1788ab7 to 5019972 Compare February 3, 2025 21:11
@tenthirtyam tenthirtyam marked this pull request as ready for review February 3, 2025 21:12
@tenthirtyam tenthirtyam changed the title fix: update alpine example refactor: update alpine example Feb 3, 2025
Copy link
Copy Markdown
Collaborator

@tenthirtyam tenthirtyam left a comment

Choose a reason for hiding this comment

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

LGTM

@tenthirtyam tenthirtyam modified the milestones: Backlog, v1.5.0 Feb 3, 2025
@tenthirtyam tenthirtyam added refactor Refactor and removed bug Bug labels Feb 3, 2025
Copy link
Copy Markdown
Contributor

@lbajolet-hashicorp lbajolet-hashicorp left a comment

Choose a reason for hiding this comment

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

LGTM!

@lbajolet-hashicorp lbajolet-hashicorp merged commit e7dcd93 into vmware:main Feb 20, 2025
@github-actions
Copy link
Copy Markdown

I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 30, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

refactor Refactor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants