So,what I do for now?
Can I can now add the new file to previous change?

On Tue, May 23, 2017 at 3:17 PM, Martin Perina <[email protected]> wrote:

> You cannot, but for future you need have the same Change-Id for a patch,
> this is how gerrit identifies patches.
>
> On Tue, May 23, 2017 at 11:43 AM, shubham dubey <[email protected]>
> wrote:
>
>> I just use 'git push origin HEAD:refs/for/master'
>> but it comes as new commit.
>>
>> How I merge both?
>>
>> On Tue, May 23, 2017 at 3:05 PM, Martin Perina <[email protected]>
>> wrote:
>>
>>> Just add a file into the patch locally and push it again to gerrit,
>>> patch will be stored as new patch set.
>>>
>>> On Tue, May 23, 2017 at 11:33 AM, shubham dubey <[email protected]>
>>> wrote:
>>>
>>>> yes I just forget to add that.
>>>> Can anyone tell me how to push the new changes to same commit?
>>>>
>>>> Thanks,
>>>> Shubham
>>>>
>>>> On Tue, May 23, 2017 at 1:27 PM, Code Review <[email protected]> wrote:
>>>>
>>>>> From Eli Mesika <[email protected]>:
>>>>>
>>>>> Eli Mesika has posted comments on this change.
>>>>>
>>>>> Change subject: core: backup flag added to storage domain static
>>>>> ......................................................................
>>>>>
>>>>>
>>>>> Patch Set 1:
>>>>>
>>>>> Did you forgot to add the relevant upgrade script that added that
>>>>> field ????
>>>>>
>>>>> --
>>>>> To view, visit https://gerrit.ovirt.org/77142
>>>>> To unsubscribe, visit https://gerrit.ovirt.org/settings
>>>>>
>>>>> Gerrit-MessageType: comment
>>>>> Gerrit-Change-Id: I1453b52f4330d0e5cb12d8fa08426fd68f3cd329
>>>>> Gerrit-PatchSet: 1
>>>>> Gerrit-Project: ovirt-engine
>>>>> Gerrit-Branch: master
>>>>> Gerrit-Owner: shubham dubey <[email protected]>
>>>>> Gerrit-Reviewer: Eli Mesika <[email protected]>
>>>>> Gerrit-Reviewer: Greg Sheremeta <[email protected]>
>>>>> Gerrit-Reviewer: Jenkins CI
>>>>> Gerrit-Reviewer: Maor Lipchuk <[email protected]>
>>>>> Gerrit-Reviewer: Martin PeÅ™ina <[email protected]>
>>>>> Gerrit-Reviewer: gerrit-hooks <[email protected]>
>>>>> Gerrit-HasComments: No
>>>>>
>>>>
>>>>
>>>
>>
>
_______________________________________________
Devel mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/devel

Reply via email to