aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jonathan Boulle" <...@twopensource.com>
Subject Re: Review Request 17063: Update tag-release to allow patch level increments.
Date Sat, 18 Jan 2014 00:20:19 GMT


> On Jan. 17, 2014, 8:12 p.m., Maxim Khutornenko wrote:
> > build-support/release/tag-release, line 70
> > <https://reviews.apache.org/r/17063/diff/1/?file=427256#file427256line70>
> >
> >     Should we even support patching off of master? What if there were interim commits
past minor/major release that are adding features that don't qualify for a patch? Should we
rather do hotfixing off of a tagged release with reverse integration into master?
> 
> Bill Farner wrote:
>     +1, i could be overthinking, but i anticipated patch releases being ~always cherry-picking.
 So far i've been committing to master, cherry-picking/backporting.

Can you elaborate on the process for this? (or confirm that the below is correct):
- Check out latest patch at the given minor level, by tag (e.g. "git checkout 0.4.2")
- Cherry-pick desired commits from master (e.g. "git cherry-pick abcd1234")
- Create new tag (e.g. "git tag -u `git config user.email` -s -m "aurora-0.4.3 release." 0.4.3")
- Profit

Could we codify this in releases.md or similar..?


- Jonathan


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17063/#review32193
-----------------------------------------------------------


On Jan. 17, 2014, 7:24 p.m., Brian Wickman wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/17063/
> -----------------------------------------------------------
> 
> (Updated Jan. 17, 2014, 7:24 p.m.)
> 
> 
> Review request for Aurora and Kevin Sweeney.
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Update tag-release to allow patch level increments.
> 
> (just now noticed I branched off the status checker change -- ignore that for now.)
> 
> 
> Diffs
> -----
> 
>   build-support/release/tag-release 90dc5da2fcb515359174d289d69cac01009f6014 
>   src/main/python/apache/aurora/executor/common/status_checker.py efffa8fda06afb830c5f65fb6eb15f4510225e96

> 
> Diff: https://reviews.apache.org/r/17063/diff/
> 
> 
> Testing
> -------
> 
> Nope
> 
> 
> Thanks,
> 
> Brian Wickman
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message