Skip to content

Default bundles to schemaVersion 1.0.1#2583

Merged
carolynvs merged 1 commit intogetporter:mainfrom
carolynvs:fix-manifest-schemaVersion
Feb 21, 2023
Merged

Default bundles to schemaVersion 1.0.1#2583
carolynvs merged 1 commit intogetporter:mainfrom
carolynvs:fix-manifest-schemaVersion

Conversation

@carolynvs
Copy link
Copy Markdown
Member

@carolynvs carolynvs commented Feb 16, 2023

What does this change

Bump the default version of bundles created with porter create to 1.0.1 with the schemaType set.

Because I forgot to update manifest.SupportedSchemaVersions in #2428 this means that people can't use Bundle schemaVersion 1.0.1 as documented in 1.0.8 and it's really available in 1.0.9 (to be released after this is merged). They can still set schemaType, and porter build doesn't fail, so it did fix the original bug. It's just that if the user bumps the schemaVersion on their porter.yaml porter build will then fail validation. Sorry that's pretty sloppy... 🤦‍♀️

What issue does it fix

Follow-up to #2428

Notes for the reviewer

N/A

Checklist

  • Did you write tests?
  • Did you write documentation?
  • Did you change porter.yaml or a storage document record? Update the corresponding schema file.
  • If this is your first pull request, please add your name to the bottom of our Contributors list. Thank you for making Porter better! 🙇‍♀️

Reviewer Checklist

  • Comment with /azp run test-porter-release if a magefile or build script was modified
  • Comment with /azp run porter-integration if it's a non-trivial PR

* Bump the default version of bundles created with `porter create` to 1.0.1 with the schemaType set.
* Fix the allowed schema versions for bundle manifests, it wasn't including 1.0.1
* Fix documentation for which version of Porter supports manifest schemaVersion 1.0.1

Signed-off-by: Carolyn Van Slyck <me@carolynvanslyck.com>
@carolynvs carolynvs force-pushed the fix-manifest-schemaVersion branch from 9ced54f to dc03f14 Compare February 16, 2023 21:17
@carolynvs carolynvs marked this pull request as ready for review February 21, 2023 18:23
@carolynvs carolynvs merged commit 38e4a62 into getporter:main Feb 21, 2023
@carolynvs carolynvs deleted the fix-manifest-schemaVersion branch February 21, 2023 18:25
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.

3 participants