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 artifact names #103

Merged
merged 13 commits into from
Nov 14, 2023
Merged

Fix artifact names #103

merged 13 commits into from
Nov 14, 2023

Conversation

jimmykarily
Copy link
Contributor

@jimmykarily jimmykarily commented Oct 31, 2023

mauromorales and others added 2 commits October 31, 2023 16:42
Signed-off-by: Dimitris Karakasilis <[email protected]>
Signed-off-by: Dimitris Karakasilis <[email protected]>
Copy link

netlify bot commented Oct 31, 2023

Deploy Preview for kairos-io ready!

Name Link
🔨 Latest commit 200b0a0
🔍 Latest deploy log https://app.netlify.com/sites/kairos-io/deploys/65480afc63f1ec000850f3e7
😎 Deploy Preview https://deploy-preview-103--kairos-io.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Signed-off-by: Dimitris Karakasilis <[email protected]>
Signed-off-by: Dimitris Karakasilis <[email protected]>
Signed-off-by: Dimitris Karakasilis <[email protected]>
Signed-off-by: Dimitris Karakasilis <[email protected]>
Signed-off-by: Dimitris Karakasilis <[email protected]>
Signed-off-by: Dimitris Karakasilis <[email protected]>
Signed-off-by: Dimitris Karakasilis <[email protected]>
Signed-off-by: Dimitris Karakasilis <[email protected]>
@jimmykarily jimmykarily marked this pull request as ready for review November 2, 2023 10:00
@jimmykarily jimmykarily self-assigned this Nov 2, 2023
hugo.toml Outdated
Comment on lines 129 to 130
exampleImageCoreTag = "leap-15.5-core-amd64-generic-v2.4.1"
exampleImageStandardTag = "leap-15.5-core-amd64-generic-v2.4.1-k3sv1.27.3-k3s1"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
exampleImageCoreTag = "leap-15.5-core-amd64-generic-v2.4.1"
exampleImageStandardTag = "leap-15.5-core-amd64-generic-v2.4.1-k3sv1.27.3-k3s1"
exampleImageCoreTag = "leap-15.5-core-amd64-generic-v2.4.1"
exampleImageStandardTag = "leap-15.5-standard-amd64-generic-v2.4.1-k3sv1.27.3-k3s1"

Standard one is incorrect here, but overall, I would prefer not to do this here since it will keep the versions of the examples pinned. I think it would be better to build them using the mechanism we already have with the version.

"{{< registryURL >}}/{{< exampleImage >}}:leap-15.5-core-amd64-generic-{{< kairosVersion >}}"

Or even better, maybe introduce another function which itself does the code above and can be used like so:

"{{< registryURL >}}/{{< exampleStandardImage >}}

maybe even go as far as

{{< exampleStandardImage >}}

Signed-off-by: Mauro Morales <[email protected]>
Signed-off-by: Mauro Morales <[email protected]>
Signed-off-by: Mauro Morales <[email protected]>
@mauromorales mauromorales merged commit 40651f0 into main Nov 14, 2023
@mauromorales mauromorales deleted the fix-artifact-names branch November 14, 2023 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants