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=shortlog; >>>>>>>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+Ide >>>>>>>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 >>>>>> >>>> >>> >> >