On 02/05/2015 09:58 AM, MICHON Anthony wrote: > Thanks for your response > I ttried to rebase to get rid of the previously commit but when I do a git > review it's rejected because no common ancestry was found > Is it any way to rewind to a fresh state, ie delete abandoned review ?
You should be able to "git rebase -i" your patch and remove the abandoned patch from it. Or create the patch from scratch again using the same ChangeID and run git review, Andreas > -----Message d'origine----- > De : Clark Boylan [mailto:[email protected]] > Envoyé : mercredi 4 février 2015 17:03 > À : [email protected] > Objet : Re: [OpenStack-Infra] Code Review needs another verified > > > > On Wed, Feb 4, 2015, at 07:51 AM, MICHON Anthony wrote: >> Hi all >> >> We committed some change on a newly created project and the review >> (https://review.openstack.org/#/c/147182/ >> ) needs another +1 verified, Jenkins gave +1 but how give another +1 ? >> It seems that others projects do not need a +2 verified to pass to the >> next step (example here : https://review.openstack.org/#/c/140517/) >> Does it missing some configuration somewhere, here are our Here are >> the project-config files : >> https://review.openstack.org/#/c/131989/ >> > All changes require +2 verified, +2 Code Review, and +1 Workflow in order to > merge. In your example [0] the green checkmarks signify that this is the > case. At a quick glance your job config looks fine. > > I believe the reason your change is not going into the gate queue and getting > a +2 verified then merging is this change depends on an abandoned change [1] > making it unmergable. I would just rebase your change to remove this > dependency then it should work as expected. > > [0] https://review.openstack.org/#/c/140517/ > [1] https://review.openstack.org/#/c/147121/ > > Clark > > _______________________________________________ > OpenStack-Infra mailing list > [email protected] > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra > > _______________________________________________ > OpenStack-Infra mailing list > [email protected] > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra > -- Andreas Jaeger aj@{suse.com,opensuse.org} Twitter/Identica: jaegerandi SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Jane Smithard, Jennifer Guild, Dilip Upmanyu, Graham Norton, HRB 21284 (AG Nürnberg) GPG fingerprint = 93A3 365E CE47 B889 DF7F FED1 389A 563C C272 A126 _______________________________________________ OpenStack-Infra mailing list [email protected] http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra
