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

Document release process. #293

Merged
merged 93 commits into from
Aug 5, 2014
Merged

Document release process. #293

merged 93 commits into from
Aug 5, 2014

Conversation

tomprince
Copy link
Contributor

Fixes #292.

This is mostly an initial sketch for discussion.

This also at least partially addresses #86, #87, #126, #277.

@itamarst
Copy link
Contributor

Basically I suggest focusing this down to whatever we will need for a very limited 0.1.


Create a zfs pool. The following create a pool as a file in ``/``::

truncate -s 2G /flocker-pool
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This needs a better location. It should probably match what the vagrant image does.

@tomprince tomprince added WIP and removed WIP labels Jul 16, 2014
4. Make sure all the tests pass on buildbot.
Go to the `buildbot <http://build.clusterhq.com/boxes-flocker>`_ and force a build on the just create branch.
XXX: buildbot needs to not merge forward on release branches
5. Do the acceptance tests. (https://github.com/ClusterHQ/flocker/issues/315)
Copy link
Contributor

Choose a reason for hiding this comment

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

I would really like these to be automated, rather than manual, but we may not have time for that...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Even if it is automated, it probably won't be run by buildbot in time for 0.1, so this would just be run those tests, in that case.

@tomprince tomprince added WIP and removed review labels Jul 16, 2014
@wallrj
Copy link
Contributor

wallrj commented Jul 31, 2014

Ok. Sorry for my slow progress on this. But I think I've now been through the process from start to finish and addressed all the earlier (and my own) code review comments.

I think it's good enough to do the first release. So I recommend that we merge it.

If there are any missing or incorrect steps we can address those at the time of the release.

@wallrj wallrj added review and removed in progress labels Jul 31, 2014
@wallrj wallrj added this to the Release 0.1 milestone Jul 31, 2014
@wallrj wallrj self-assigned this Jul 31, 2014
@wallrj
Copy link
Contributor

wallrj commented Aug 4, 2014

The installation documentation in this branch refers to flocker-release is that meant to be clusterhq-release or vice-versa.

Also worth adding a link and a sentence to these new installation instructions in the (also new) flocker tutorial/requirements step. The tutorial doesn't currently explain how to install flocker-cli.

If you're interested in developing Flocker (as opposed to simply using it) see :doc:`contributing`.
Installing flocker-node
=======================
.. note:: If you're interested in developing Flocker (as opposed to simply using it) see :doc:`contributing`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Move this note above the headline, to previous section.

@tomprince
Copy link
Contributor Author

flocker-release here should be clusterhq-release I think I changed my mind what it should be called after starting this, and forgot to go back and change it.

@itamarst
Copy link
Contributor

itamarst commented Aug 4, 2014

At this point I think we've spent way too much time with this branch open. Address remaining comments and then merge, and we can do additional cleanups as a second pass as we improve documentation separately (e.g. #87) or find issues when we do the test release.

@itamarst itamarst added accepted and removed review labels Aug 4, 2014
@wallrj
Copy link
Contributor

wallrj commented Aug 5, 2014

Ok. I addressed those final comments. Merging.

wallrj added a commit that referenced this pull request Aug 5, 2014
Document the flocker release process

Author: @tomprince 
Reviewers: @wallrj, @itamarst 

Fixes #292
@wallrj wallrj merged commit c2aaacf into master Aug 5, 2014
@wallrj wallrj deleted the release-process-292 branch August 5, 2014 09:13
wallrj added a commit that referenced this pull request Aug 11, 2014
Release documentation improvements resulting from the 0.0.6 practice release

Author: @wallrj
Reviewer: @adamtheturtle 

Refs #293
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Document the release process.
5 participants