Thanks Marcus. I am not aware of this convention, will remember that next time when I do the merge.
-min On 3/13/14 10:30 PM, "Marcus" <shadow...@gmail.com> wrote: >Min, in looking at this branch merge, I need to be reminded whether we >are supposed to squash feature branches when they come in, or preserve >history. It's nice to preserve history, but it's a lot easier to undo >a squashed merge. > >On Thu, Mar 13, 2014 at 5:56 PM, Min Chen <min.c...@citrix.com> wrote: >> IAM branch is now merged to master. >> >> Thanks >> -min >> >> On 3/13/14 10:13 AM, "Min Chen" <min.c...@citrix.com> wrote: >> >>>Since we haven't heard of any objections to this merge for 3 days, I am >>>going to merge it to master today. >>> >>>Thanks >>>-min >>> >>>On 3/11/14 12:23 PM, "Hugo Trippaers" <h...@trippaers.nl> wrote: >>> >>>> >>>>On 11 mrt. 2014, at 19:52, Min Chen <min.c...@citrix.com> wrote: >>>> >>>>> Also, have already run FingBugs on our branch and addressed all new >>>>> findings introduced by our branch. >>>> >>>>Awesome! :-) >>>> >>>>> >>>>> Thanks. >>>>> -min >>>>> >>>>> On 3/10/14 7:33 PM, "Min Chen" <min.c...@citrix.com> wrote: >>>>> >>>>>> No new jar dependencies. >>>>>> >>>>>> -min >>>>>> >>>>>> Sent from my iPhone >>>>>> >>>>>>> On Mar 10, 2014, at 7:22 PM, "Chiradeep Vittal" >>>>>>> <chiradeep.vit...@citrix.com> wrote: >>>>>>> >>>>>>> Any new jar dependencies? >>>>>>> >>>>>>>> On 3/10/14, 11:34 AM, "Min Chen" <min.c...@citrix.com> wrote: >>>>>>>> >>>>>>>> Hi, >>>>>>>> >>>>>>>> Prachi and I would like to merge CloudStack Identity and Access >>>>>>>> Management(IAM) plugin services to the master branch. Development >>>>>>>>for >>>>>>>> this effort has been done by Prachi and me on ACS rbac branch >>>>>>>> >>>>>>>> >>>>>>>>(https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlo >>>>>>>>g; >>>>>>>>h >>>>>>>>=r >>>>>>>> ef >>>>>>>> s/heads/rbac). >>>>>>>> Checklists for the merge: >>>>>>>> 1. JIRA ticket: >>>>>>>>https://issues.apache.org/jira/browse/CLOUDSTACK-5920. >>>>>>>> 2. Functional Specs: >>>>>>>> >>>>>>>> >>>>>>>>https://cwiki.apache.org/confluence/display/CLOUDSTACK/CloudStack+I >>>>>>>>de >>>>>>>>n >>>>>>>>ti >>>>>>>> ty >>>>>>>> +and+Access+Management+%28IAM%29+Plugin. We have proposed this >>>>>>>>feature >>>>>>>> back in Jan, and accommodated all the feedbacks in our >>>>>>>>implementation. >>>>>>>> 3. Unit tests for the feature are available at: >>>>>>>> services/iam/server/test >>>>>>>> (for iam server) and services/iam/plugin/test (for iam plugin). >>>>>>>> 4. Marvin integration tests for the feature are available at: >>>>>>>> test/integration/smoke/test_vm_iam.py. >>>>>>>> 5. Branch has been rebased with master branch up to commit >>>>>>>> 63e3eea7905e22cab9466b28a2ab2a80b586aeed. >>>>>>>> 6. RAT test has been passed. >>>>>>>> >>>>>>>> Thanks. >>>>>>>> -min >>>>>>> >>>>> >>>> >>> >>
