Skip to content

Conversation

@yharish991-brcm
Copy link
Collaborator

This PR

  • Adds the ability to enable windows2025 and noble stemcell download when downloading a product which specifies these stemcells as the dependency
  • Fixes a couple of small bugs to enable stemcell downloads from non-pivnet sources.

)

var metadataRegexp = regexp.MustCompile(`^(\.\/)?metadata/.*\.yml`)
var metadataRegexp = regexp.MustCompile(`^(.*\/)?metadata/.*\.yml`)
Copy link
Collaborator Author

@yharish991-brcm yharish991-brcm Sep 17, 2025

Choose a reason for hiding this comment

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

previously this is was only looking for file under metadata/ directory, but when the downloaded product is extracted it can be in a sub directory ex: download-product/product-name/metadata/, this change is to support that case.. the change will continue to support looking for file under metadata/, so this is non-breaking

Copy link
Member

Choose a reason for hiding this comment

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

agreed

Copy link
Member

@drich10 drich10 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 to me!

)

var metadataRegexp = regexp.MustCompile(`^(\.\/)?metadata/.*\.yml`)
var metadataRegexp = regexp.MustCompile(`^(.*\/)?metadata/.*\.yml`)
Copy link
Member

Choose a reason for hiding this comment

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

agreed

@yharish991-brcm yharish991-brcm merged commit 0dc8ae9 into main Sep 19, 2025
1 check passed
@yharish991-brcm yharish991-brcm deleted the windows2025Stemcell branch September 19, 2025 15:44
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