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

Add RELEASE.toml #556

Merged
merged 1 commit into from
Dec 16, 2019
Merged

Add RELEASE.toml #556

merged 1 commit into from
Dec 16, 2019

Conversation

zmrow
Copy link
Contributor

@zmrow zmrow commented Nov 21, 2019

The purpose of this file is to provide a single point of truth for
anything that needs to reference the current version of Thar. It also
contains the list of migrations for each version update.

Issue #, if available:
#537

Description of changes:

  • Add RELEASE.toml to the tree

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

RELEASE.toml Outdated Show resolved Hide resolved
RELEASE.toml Outdated Show resolved Hide resolved
@zmrow
Copy link
Contributor Author

zmrow commented Nov 21, 2019

Fixed my typos

@sam-aws
Copy link
Contributor

sam-aws commented Nov 21, 2019

reference the current version of Thar.

To clarify, my understanding is that this version is "when you build this what version should it think it is" and not "this is the latest version of Thar" which would only be asserted by a git tag. Is that right?

tjkirch
tjkirch previously approved these changes Nov 21, 2019
Copy link
Contributor

@tjkirch tjkirch left a comment

Choose a reason for hiding this comment

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

this version is "when you build this what version should it think it is" and not "this is the latest version of Thar" which would only be asserted by a git tag. Is that right?

If you're making a distinction between "latest released version" and "next released version", then I think it's right - this will be the version upon the next release. The release will build using this version, and then we'll update the version, so that we can do the following release whenever we need.

@@ -0,0 +1,6 @@
version: "0.2.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: I'd prefer a header here, maybe [os] or [main] or ...?

Copy link
Contributor

Choose a reason for hiding this comment

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

Why a header? The naming is going to be a bikeshed, and it's easier to access the data without... no?

Copy link
Contributor

Choose a reason for hiding this comment

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

It just feels weird to have no header.

That's why it's a nit 😸

RELEASE.toml Outdated
[[migrations]]
from: "0.1.6"
to: "0.2.0"
names: ["borkseed", "host-containers-version"]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@sam-aws , @tjkirch , @iliana

What exactly do these names need to be? The names of the compiled binaries? I'm pretty sure they're currently wrong.

Copy link
Contributor

Choose a reason for hiding this comment

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

These need to be of the form migrate_VERSION_NAME, eg. migrate_0.1_borkseed.
Once we fold the datastore version into the image version we can update the migration tools to recognise migrate_0.2.0_borkseed instead.

@tjkirch tjkirch dismissed their stale review December 6, 2019 18:25

Need updates for migration names

The purpose of this file is to provide a single point of truth for
anything that needs to reference the current version of Thar. It also
contains the list of migrations for each version update.
@zmrow
Copy link
Contributor Author

zmrow commented Dec 12, 2019

Added datastore_version and corrected the migration names

@zmrow zmrow requested a review from bcressey December 12, 2019 22:36
@zmrow
Copy link
Contributor Author

zmrow commented Dec 12, 2019

@bcressey How should we reconcile this with the version that lives in packages/release/release.spec? I think this deserves a bit of thought in regards to our release process.

Perhaps these remain in conflict (with RELEASE.toml being the "next" version) until we are ready to do a "real" release?

Copy link
Contributor

@tjkirch tjkirch left a comment

Choose a reason for hiding this comment

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

I think we should start here and iterate!

@zmrow zmrow merged commit 1a04398 into develop Dec 16, 2019
@zmrow zmrow deleted the releasetoml branch December 16, 2019 19:23
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