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

Allow ISO Storage repository to fallback to default SR if unspecified #109

Closed
wants to merge 197 commits into from

Conversation

ddelnano
Copy link

#63 missed an update to the config structures.

Testing

  • Verify that default iso fallback works. Currently this is failing because the config structures are still requiring that sr_iso_name is defined as seen below

ddelnano and others added 30 commits September 13, 2020 01:22
…os VMs can run kickstarter (albeit with errors)
Get the packer builder compiling and ensure that it can create terraform compatible VM templates
Remove unused replace directive in go.mod file
Add start of centos 8 and ubuntu 20.04 examples
ddelnano and others added 28 commits January 21, 2023 00:24
Remove hashicorps gpg github action to one that doesn't collide with Github's
(cherry picked from commit 2b2ba8540d1acfa0884e8db1367b32495bc39259)
…o-be-resilent-to-new-releases

Update the ubuntu example to be resilient to ISO releases
…-example-to-be-resilent-to-new-releases

Revert "Update the ubuntu example to be resilient to ISO releases"
…uploaded-each-time

Prevent ISO used in iso builder from being uploaded if it already exists
Update ubuntu example to be resilient to upstream minor releases
Signed-off-by: AtaxyaNetwork <[email protected]>
Looks like the consts got moved into common
Solves TestBuilderPrepare_InvalidKey failing, since we weren't capturing
it's errors before.
This aligns the Xenserver plugin to being a bit more inline with what
Packer > 1.6.0 is expecting, since packer now simply ignores the
iso_checksum_type (it's supposed to error out but that code path isn't
working right now because we don't set PluginType in the configs. The
unit tests have been altered to reflect this reality.

Note that this isn't a comprehensive change; the config still has the
inert ISOChecksumType, and there's probably a laundry list of other
things that needs to be looked at, For now though, we have working
unit tests again.

Documentation from SDK has been aligned for iso_checksum
…e_and_fix_tests

Remove iso checksum type and fix tests
@ddelnano ddelnano closed this Apr 14, 2023
@ddelnano ddelnano deleted the ddelnano/sohonetlabs/iso-sr branch April 14, 2023 13:47
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.