meta-openembedded patchwork git hook

meta-openembedded patchwork git hook

[Paul Eggleton]

Hi folks,

Martin just brought to my attention that the git hook for the 
meta-openembedded repository that's supposed to update patchwork still isn't 
working and hasn't done so for a very long time. I thought we'd fixed this so 
that merged patches get marked as "Accepted", but at least for meta-
openembedded it seems like that's not the case.

Jose / Michael - can you look into this if you haven't already? Please let me 
know if you need any help.

Thanks,
Paul

-- 
Paul Eggleton
Intel Open Source Technology Centre

Re: meta-openembedded patchwork git hook

[Tom Rini]

On Wed, Nov 30, 2016 at 06:52:46AM +1300, Paul Eggleton wrote:

> Hi folks,
> 
> Martin just brought to my attention that the git hook for the 
> meta-openembedded repository that's supposed to update patchwork still isn't 
> working and hasn't done so for a very long time. I thought we'd fixed this so 
> that merged patches get marked as "Accepted", but at least for meta-
> openembedded it seems like that's not the case.
> 
> Jose / Michael - can you look into this if you haven't already? Please let me 
> know if you need any help.

Since I brought this up on IRC, I wanted to chime in here.  First, if
you want to use top-of-tree patchwork you'll also want to grab
https://patchwork.ozlabs.org/bundle/stephenfin/post-receive%20fixes/
(and fwiw, I also have been setting the DJANGO part to 'dev' rather than
production for my use-case).  Second, I think part of the problem is
that there are just some cases where the hashing fails, which for me is
around 10% of the time.  Depending on how much one uses bundles and
tracks what ends up being committed this may or may not be a problem.
And I'm sure upstream patchwork would appreciate any changes to improve
hashing success :)

-- 
Tom

Re: meta-openembedded patchwork git hook

[Jose Lamego]

[-- Attachment #1.1: Type: text/plain, Size: 1675 bytes --]



On 11/29/2016 02:43 PM, Tom Rini wrote:
> On Wed, Nov 30, 2016 at 06:52:46AM +1300, Paul Eggleton wrote:
> 
>> Hi folks,
>>
>> Martin just brought to my attention that the git hook for the 
>> meta-openembedded repository that's supposed to update patchwork still isn't 
>> working and hasn't done so for a very long time. I thought we'd fixed this so 
>> that merged patches get marked as "Accepted", but at least for meta-
>> openembedded it seems like that's not the case.
>>
>> Jose / Michael - can you look into this if you haven't already? Please let me 
>> know if you need any help.

The git hook was configured to update OE-core only, Michael will help us
to include meta-openembedded.

> 
> Since I brought this up on IRC, I wanted to chime in here.  First, if
> you want to use top-of-tree patchwork you'll also want to grab
> https://patchwork.ozlabs.org/bundle/stephenfin/post-receive%20fixes/


Hi Tom, nice meeting you. OE-Patchwork is using a fork in [1] from
freedesktop.org's patchwork at [2]. I will look at the hook fixes you
mentioned at the first chance, thanks for the tip.

> (and fwiw, I also have been setting the DJANGO part to 'dev' rather than
> production for my use-case).  Second, I think part of the problem is
> that there are just some cases where the hashing fails, which for me is
> around 10% of the time.  Depending on how much one uses bundles and
> tracks what ends up being committed this may or may not be a problem.
> And I'm sure upstream patchwork would appreciate any changes to improve
> hashing success :)
> 

Regards
Jose

-- 
Jose Lamego | OTC Embedded Platforms & Tools | GDC


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 501 bytes --]

Re: meta-openembedded patchwork git hook

[Khem Raj]

> On Nov 29, 2016, at 2:34 PM, Jose Lamego <jose.a.lamego@linux.intel.com> wrote:
> 
> 
> 
> On 11/29/2016 02:43 PM, Tom Rini wrote:
>> On Wed, Nov 30, 2016 at 06:52:46AM +1300, Paul Eggleton wrote:
>> 
>>> Hi folks,
>>> 
>>> Martin just brought to my attention that the git hook for the 
>>> meta-openembedded repository that's supposed to update patchwork still isn't 
>>> working and hasn't done so for a very long time. I thought we'd fixed this so 
>>> that merged patches get marked as "Accepted", but at least for meta-
>>> openembedded it seems like that's not the case.
>>> 
>>> Jose / Michael - can you look into this if you haven't already? Please let me 
>>> know if you need any help.
> 
> The git hook was configured to update OE-core only, Michael will help us
> to include meta-openembedded.
> 

There are more repos using patchwork beyond OE-core and meta-opemebedded
please enable it for all of them  as well.

>> 
>> Since I brought this up on IRC, I wanted to chime in here.  First, if
>> you want to use top-of-tree patchwork you'll also want to grab
>> https://patchwork.ozlabs.org/bundle/stephenfin/post-receive%20fixes/
> 
> 
> Hi Tom, nice meeting you. OE-Patchwork is using a fork in [1] from
> freedesktop.org's patchwork at [2]. I will look at the hook fixes you
> mentioned at the first chance, thanks for the tip.
> 
>> (and fwiw, I also have been setting the DJANGO part to 'dev' rather than
>> production for my use-case).  Second, I think part of the problem is
>> that there are just some cases where the hashing fails, which for me is
>> around 10% of the time.  Depending on how much one uses bundles and
>> tracks what ends up being committed this may or may not be a problem.
>> And I'm sure upstream patchwork would appreciate any changes to improve
>> hashing success :)
>> 
> 
> Regards
> Jose
> 
> -- 
> Jose Lamego | OTC Embedded Platforms & Tools | GDC
> 
> -- 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel

Re: meta-openembedded patchwork git hook

[Jose Lamego]

[-- Attachment #1.1: Type: text/plain, Size: 2480 bytes --]



On 11/29/2016 07:50 PM, Khem Raj wrote:
> 
>> On Nov 29, 2016, at 2:34 PM, Jose Lamego <jose.a.lamego@linux.intel.com> wrote:
>>
>>
>>
>> On 11/29/2016 02:43 PM, Tom Rini wrote:
>>> On Wed, Nov 30, 2016 at 06:52:46AM +1300, Paul Eggleton wrote:
>>>
>>>> Hi folks,
>>>>
>>>> Martin just brought to my attention that the git hook for the 
>>>> meta-openembedded repository that's supposed to update patchwork still isn't 
>>>> working and hasn't done so for a very long time. I thought we'd fixed this so 
>>>> that merged patches get marked as "Accepted", but at least for meta-
>>>> openembedded it seems like that's not the case.
>>>>
>>>> Jose / Michael - can you look into this if you haven't already? Please let me 
>>>> know if you need any help.
>>
>> The git hook was configured to update OE-core only, Michael will help us
>> to include meta-openembedded.
>>
> 
> There are more repos using patchwork beyond OE-core and meta-opemebedded
> please enable it for all of them  as well.
> 

Now all of the projects are enabled to receive automatic updates from
the git-hook (Thanks Michael). Please let me know if any undesired
behavior is detected.

>>>
>>> Since I brought this up on IRC, I wanted to chime in here.  First, if
>>> you want to use top-of-tree patchwork you'll also want to grab
>>> https://patchwork.ozlabs.org/bundle/stephenfin/post-receive%20fixes/
>>
>>
>> Hi Tom, nice meeting you. OE-Patchwork is using a fork in [1] from
>> freedesktop.org's patchwork at [2]. I will look at the hook fixes you
>> mentioned at the first chance, thanks for the tip.
>>
>>> (and fwiw, I also have been setting the DJANGO part to 'dev' rather than
>>> production for my use-case).  Second, I think part of the problem is
>>> that there are just some cases where the hashing fails, which for me is
>>> around 10% of the time.  Depending on how much one uses bundles and
>>> tracks what ends up being committed this may or may not be a problem.
>>> And I'm sure upstream patchwork would appreciate any changes to improve
>>> hashing success :)
>>>
>>
>> Regards
>> Jose
>>
>> -- 
>> Jose Lamego | OTC Embedded Platforms & Tools | GDC
>>
>> -- 
>> _______________________________________________
>> Openembedded-devel mailing list
>> Openembedded-devel@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> 

-- 
Jose Lamego | OTC Embedded Platforms & Tools | GDC


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 501 bytes --]

Re: meta-openembedded patchwork git hook

[Martin Jansa]

[-- Attachment #1: Type: text/plain, Size: 5241 bytes --]

On Wed, Nov 30, 2016 at 12:41:20PM -0600, Jose Lamego wrote:
> 
> 
> On 11/29/2016 07:50 PM, Khem Raj wrote:
> > 
> >> On Nov 29, 2016, at 2:34 PM, Jose Lamego <jose.a.lamego@linux.intel.com> wrote:
> >>
> >>
> >>
> >> On 11/29/2016 02:43 PM, Tom Rini wrote:
> >>> On Wed, Nov 30, 2016 at 06:52:46AM +1300, Paul Eggleton wrote:
> >>>
> >>>> Hi folks,
> >>>>
> >>>> Martin just brought to my attention that the git hook for the 
> >>>> meta-openembedded repository that's supposed to update patchwork still isn't 
> >>>> working and hasn't done so for a very long time. I thought we'd fixed this so 
> >>>> that merged patches get marked as "Accepted", but at least for meta-
> >>>> openembedded it seems like that's not the case.
> >>>>
> >>>> Jose / Michael - can you look into this if you haven't already? Please let me 
> >>>> know if you need any help.
> >>
> >> The git hook was configured to update OE-core only, Michael will help us
> >> to include meta-openembedded.
> >>
> > 
> > There are more repos using patchwork beyond OE-core and meta-opemebedded
> > please enable it for all of them  as well.
> > 
> 
> Now all of the projects are enabled to receive automatic updates from
> the git-hook (Thanks Michael). Please let me know if any undesired
> behavior is detected.

Still doesn't mark anything as accepted:

OE @ ~/meta-openembedded $ git push origin master
Counting objects: 293, done.
Delta compression using up to 8 threads.
Compressing objects: 100% (257/257), done.
Writing objects: 100% (293/293), 48.83 KiB | 0 bytes/s, done.
Total 293 (delta 179), reused 68 (delta 18)
remote: E: failed to find patch for rev efd3696e70a6603f1a45faa4a172433514f0a487.
remote: E: failed to find patch for rev 8c720146b718aae3bc194621f3e095cf11ecb2ed.
remote: E: failed to find patch for rev fc2fc9097b2fa437d4af0338e418692b4dd6c989.
remote: E: failed to find patch for rev 533826b7652a2cf86ca7b665c25b98c89afc266e.
remote: E: failed to find patch for rev 6afb85d29286aaa4cdd892460f6adf68912917a5.
remote: E: failed to find patch for rev a7f3ec87f957ff35b8a853cd0e04c6b18bac0ece.
remote: E: failed to find patch for rev 877e52fd75c7d3b88add94691df3beb5e0c54529.
remote: E: failed to find patch for rev 2f6e45baba961629bfe023599144be6cfcd45728.
remote: E: failed to find patch for rev 86fed80d80bc33f0a3cdbf719c00ee5f05ed4d19.
remote: E: failed to find patch for rev 41d7bac5d50091cab835eac0a5eaa6c431fb2023.
remote: E: failed to find patch for rev a85e12928131fce057f777daa76a7f0c75cffa14.
remote: E: failed to find patch for rev 45837e6c567b1b9ff9d152a7e2a752488d313455.
remote: E: failed to find patch for rev 119e75e48dbf0539b4e440417901458ffff79b38.
remote: E: failed to find patch for rev a4251575f41fa5aafe5072e562497b05c11b6702.
remote: E: failed to find patch for rev b6ea2c0bacb21b83bae3605ef4da080de0cad15e.
remote: E: failed to find patch for rev 4580e5ae201b320ed407eae26f58655b8768b387.
remote: E: failed to find patch for rev c476219e845934f767927e81fe60dbaa157df9ee.
remote: E: failed to find patch for rev 0288b204f3c12af1561218fa0cfcdfb9a5508b47.
remote: E: failed to find patch for rev 5596b52d64c30d8e66ec6e379cf69a9fc9407fb1.
remote: E: failed to find patch for rev 24e51a6a106bc10a8838b23e4bf9ea6304314bca.
remote: E: failed to find patch for rev aac6bf8bf911c3f14ff9dce08f0f011838f5a426.
remote: I: 0 patch(es) updated to state Accepted.
remote: Sending notification emails to: openembedded-commits@lists.openembedded.org.
To git.openembedded.org:meta-openembedded
   761639b..aac6bf8  master -> master

> >>>
> >>> Since I brought this up on IRC, I wanted to chime in here.  First, if
> >>> you want to use top-of-tree patchwork you'll also want to grab
> >>> https://patchwork.ozlabs.org/bundle/stephenfin/post-receive%20fixes/
> >>
> >>
> >> Hi Tom, nice meeting you. OE-Patchwork is using a fork in [1] from
> >> freedesktop.org's patchwork at [2]. I will look at the hook fixes you
> >> mentioned at the first chance, thanks for the tip.
> >>
> >>> (and fwiw, I also have been setting the DJANGO part to 'dev' rather than
> >>> production for my use-case).  Second, I think part of the problem is
> >>> that there are just some cases where the hashing fails, which for me is
> >>> around 10% of the time.  Depending on how much one uses bundles and
> >>> tracks what ends up being committed this may or may not be a problem.
> >>> And I'm sure upstream patchwork would appreciate any changes to improve
> >>> hashing success :)
> >>>
> >>
> >> Regards
> >> Jose
> >>
> >> -- 
> >> Jose Lamego | OTC Embedded Platforms & Tools | GDC
> >>
> >> -- 
> >> _______________________________________________
> >> Openembedded-devel mailing list
> >> Openembedded-devel@lists.openembedded.org
> >> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> > 
> 
> -- 
> Jose Lamego | OTC Embedded Platforms & Tools | GDC
> 




> -- 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel


-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 201 bytes --]

Re: meta-openembedded patchwork git hook

[Tom Rini]

On Fri, Dec 02, 2016 at 09:30:09AM +0100, Martin Jansa wrote:
> On Wed, Nov 30, 2016 at 12:41:20PM -0600, Jose Lamego wrote:
> > 
> > 
> > On 11/29/2016 07:50 PM, Khem Raj wrote:
> > > 
> > >> On Nov 29, 2016, at 2:34 PM, Jose Lamego <jose.a.lamego@linux.intel.com> wrote:
> > >>
> > >>
> > >>
> > >> On 11/29/2016 02:43 PM, Tom Rini wrote:
> > >>> On Wed, Nov 30, 2016 at 06:52:46AM +1300, Paul Eggleton wrote:
> > >>>
> > >>>> Hi folks,
> > >>>>
> > >>>> Martin just brought to my attention that the git hook for the 
> > >>>> meta-openembedded repository that's supposed to update patchwork still isn't 
> > >>>> working and hasn't done so for a very long time. I thought we'd fixed this so 
> > >>>> that merged patches get marked as "Accepted", but at least for meta-
> > >>>> openembedded it seems like that's not the case.
> > >>>>
> > >>>> Jose / Michael - can you look into this if you haven't already? Please let me 
> > >>>> know if you need any help.
> > >>
> > >> The git hook was configured to update OE-core only, Michael will help us
> > >> to include meta-openembedded.
> > >>
> > > 
> > > There are more repos using patchwork beyond OE-core and meta-opemebedded
> > > please enable it for all of them  as well.
> > > 
> > 
> > Now all of the projects are enabled to receive automatic updates from
> > the git-hook (Thanks Michael). Please let me know if any undesired
> > behavior is detected.
> 
> Still doesn't mark anything as accepted:
> 
> OE @ ~/meta-openembedded $ git push origin master
> Counting objects: 293, done.
> Delta compression using up to 8 threads.
> Compressing objects: 100% (257/257), done.
> Writing objects: 100% (293/293), 48.83 KiB | 0 bytes/s, done.
> Total 293 (delta 179), reused 68 (delta 18)
> remote: E: failed to find patch for rev efd3696e70a6603f1a45faa4a172433514f0a487.
> remote: E: failed to find patch for rev 8c720146b718aae3bc194621f3e095cf11ecb2ed.
> remote: E: failed to find patch for rev fc2fc9097b2fa437d4af0338e418692b4dd6c989.
> remote: E: failed to find patch for rev 533826b7652a2cf86ca7b665c25b98c89afc266e.
> remote: E: failed to find patch for rev 6afb85d29286aaa4cdd892460f6adf68912917a5.
> remote: E: failed to find patch for rev a7f3ec87f957ff35b8a853cd0e04c6b18bac0ece.
> remote: E: failed to find patch for rev 877e52fd75c7d3b88add94691df3beb5e0c54529.
> remote: E: failed to find patch for rev 2f6e45baba961629bfe023599144be6cfcd45728.
> remote: E: failed to find patch for rev 86fed80d80bc33f0a3cdbf719c00ee5f05ed4d19.
> remote: E: failed to find patch for rev 41d7bac5d50091cab835eac0a5eaa6c431fb2023.
> remote: E: failed to find patch for rev a85e12928131fce057f777daa76a7f0c75cffa14.
> remote: E: failed to find patch for rev 45837e6c567b1b9ff9d152a7e2a752488d313455.
> remote: E: failed to find patch for rev 119e75e48dbf0539b4e440417901458ffff79b38.
> remote: E: failed to find patch for rev a4251575f41fa5aafe5072e562497b05c11b6702.
> remote: E: failed to find patch for rev b6ea2c0bacb21b83bae3605ef4da080de0cad15e.
> remote: E: failed to find patch for rev 4580e5ae201b320ed407eae26f58655b8768b387.
> remote: E: failed to find patch for rev c476219e845934f767927e81fe60dbaa157df9ee.
> remote: E: failed to find patch for rev 0288b204f3c12af1561218fa0cfcdfb9a5508b47.
> remote: E: failed to find patch for rev 5596b52d64c30d8e66ec6e379cf69a9fc9407fb1.
> remote: E: failed to find patch for rev 24e51a6a106bc10a8838b23e4bf9ea6304314bca.
> remote: E: failed to find patch for rev aac6bf8bf911c3f14ff9dce08f0f011838f5a426.
> remote: I: 0 patch(es) updated to state Accepted.
> remote: Sending notification emails to: openembedded-commits@lists.openembedded.org.
> To git.openembedded.org:meta-openembedded
>    761639b..aac6bf8  master -> master

I don't know how far apart the freedesktop.org one is from ozlabs but
when I was seeing that problem locally as part of what I was working on
it was because of some missing host deps (that are now not required).
FWIW you can pretty easily turn the hook into a script to run manually
and sh -x it to see _why_ it's not hashing out those patches.

-- 
Tom

Re: meta-openembedded patchwork git hook

[Jose Lamego]

[-- Attachment #1.1: Type: text/plain, Size: 4397 bytes --]

I'm looking into this right now. I've filed a bug for tracking:
https://bugzilla.yoctoproject.org/show_bug.cgi?id=10759


On 12/02/2016 06:29 AM, Tom Rini wrote:
> On Fri, Dec 02, 2016 at 09:30:09AM +0100, Martin Jansa wrote:
>> On Wed, Nov 30, 2016 at 12:41:20PM -0600, Jose Lamego wrote:
>>>
>>>
>>> On 11/29/2016 07:50 PM, Khem Raj wrote:
>>>>
>>>>> On Nov 29, 2016, at 2:34 PM, Jose Lamego <jose.a.lamego@linux.intel.com> wrote:
>>>>>
>>>>>
>>>>>
>>>>> On 11/29/2016 02:43 PM, Tom Rini wrote:
>>>>>> On Wed, Nov 30, 2016 at 06:52:46AM +1300, Paul Eggleton wrote:
>>>>>>
>>>>>>> Hi folks,
>>>>>>>
>>>>>>> Martin just brought to my attention that the git hook for the 
>>>>>>> meta-openembedded repository that's supposed to update patchwork still isn't 
>>>>>>> working and hasn't done so for a very long time. I thought we'd fixed this so 
>>>>>>> that merged patches get marked as "Accepted", but at least for meta-
>>>>>>> openembedded it seems like that's not the case.
>>>>>>>
>>>>>>> Jose / Michael - can you look into this if you haven't already? Please let me 
>>>>>>> know if you need any help.
>>>>>
>>>>> The git hook was configured to update OE-core only, Michael will help us
>>>>> to include meta-openembedded.
>>>>>
>>>>
>>>> There are more repos using patchwork beyond OE-core and meta-opemebedded
>>>> please enable it for all of them  as well.
>>>>
>>>
>>> Now all of the projects are enabled to receive automatic updates from
>>> the git-hook (Thanks Michael). Please let me know if any undesired
>>> behavior is detected.
>>
>> Still doesn't mark anything as accepted:
>>
>> OE @ ~/meta-openembedded $ git push origin master
>> Counting objects: 293, done.
>> Delta compression using up to 8 threads.
>> Compressing objects: 100% (257/257), done.
>> Writing objects: 100% (293/293), 48.83 KiB | 0 bytes/s, done.
>> Total 293 (delta 179), reused 68 (delta 18)
>> remote: E: failed to find patch for rev efd3696e70a6603f1a45faa4a172433514f0a487.
>> remote: E: failed to find patch for rev 8c720146b718aae3bc194621f3e095cf11ecb2ed.
>> remote: E: failed to find patch for rev fc2fc9097b2fa437d4af0338e418692b4dd6c989.
>> remote: E: failed to find patch for rev 533826b7652a2cf86ca7b665c25b98c89afc266e.
>> remote: E: failed to find patch for rev 6afb85d29286aaa4cdd892460f6adf68912917a5.
>> remote: E: failed to find patch for rev a7f3ec87f957ff35b8a853cd0e04c6b18bac0ece.
>> remote: E: failed to find patch for rev 877e52fd75c7d3b88add94691df3beb5e0c54529.
>> remote: E: failed to find patch for rev 2f6e45baba961629bfe023599144be6cfcd45728.
>> remote: E: failed to find patch for rev 86fed80d80bc33f0a3cdbf719c00ee5f05ed4d19.
>> remote: E: failed to find patch for rev 41d7bac5d50091cab835eac0a5eaa6c431fb2023.
>> remote: E: failed to find patch for rev a85e12928131fce057f777daa76a7f0c75cffa14.
>> remote: E: failed to find patch for rev 45837e6c567b1b9ff9d152a7e2a752488d313455.
>> remote: E: failed to find patch for rev 119e75e48dbf0539b4e440417901458ffff79b38.
>> remote: E: failed to find patch for rev a4251575f41fa5aafe5072e562497b05c11b6702.
>> remote: E: failed to find patch for rev b6ea2c0bacb21b83bae3605ef4da080de0cad15e.
>> remote: E: failed to find patch for rev 4580e5ae201b320ed407eae26f58655b8768b387.
>> remote: E: failed to find patch for rev c476219e845934f767927e81fe60dbaa157df9ee.
>> remote: E: failed to find patch for rev 0288b204f3c12af1561218fa0cfcdfb9a5508b47.
>> remote: E: failed to find patch for rev 5596b52d64c30d8e66ec6e379cf69a9fc9407fb1.
>> remote: E: failed to find patch for rev 24e51a6a106bc10a8838b23e4bf9ea6304314bca.
>> remote: E: failed to find patch for rev aac6bf8bf911c3f14ff9dce08f0f011838f5a426.
>> remote: I: 0 patch(es) updated to state Accepted.
>> remote: Sending notification emails to: openembedded-commits@lists.openembedded.org.
>> To git.openembedded.org:meta-openembedded
>>    761639b..aac6bf8  master -> master
> 
> I don't know how far apart the freedesktop.org one is from ozlabs but
> when I was seeing that problem locally as part of what I was working on
> it was because of some missing host deps (that are now not required).
> FWIW you can pretty easily turn the hook into a script to run manually
> and sh -x it to see _why_ it's not hashing out those patches.
> 

-- 
Jose Lamego | OTC Embedded Platforms & Tools | GDC


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 501 bytes --]

Re: meta-openembedded patchwork git hook

[Martin Jansa]

[-- Attachment #1: Type: text/plain, Size: 8022 bytes --]

On Fri, Dec 02, 2016 at 09:45:22AM -0600, Jose Lamego wrote:
> I'm looking into this right now. I've filed a bug for tracking:
> https://bugzilla.yoctoproject.org/show_bug.cgi?id=10759

This bug is already closed, but it still doesn't work, where is that
separate ticket?

It still doesn't work for meta-oe and for some reason it's also much
slower than it used to.

Took couple minutes to push just a few changes today and no patch was
updated on patchwork by the git hook, it used to take less than 1 minute.

OE @ ~/meta-openembedded $ git push origin master
Counting objects: 190, done.
Delta compression using up to 8 threads.
Compressing objects: 100% (163/163), done.
Writing objects: 100% (190/190), 23.37 KiB | 0 bytes/s, done.
Total 190 (delta 122), reused 11 (delta 7)
remote: E: failed to find patch for rev 5f85256b46ba2d89bf27f9d3edd607c4b59cf389.
remote: E: failed to find patch for rev 142faee1734300a041fe16265cb63cd17fb742ae.
remote: E: failed to find patch for rev 3237c650e79c9046d0e1b94568a45d5c6876dbd2.
remote: E: failed to find patch for rev 9aace0c5667ab910078cfdd1e388e386f106c074.
remote: E: failed to find patch for rev a1bdd5f40e15312bb4891cfa2e0d67d590051d00.
remote: E: failed to find patch for rev 3d998f3745b90c6b4db5eaa739e4f976b5d5eef2.
remote: E: failed to find patch for rev 9cd05c16510cb0b6a09f72e5c7488d021a55440b.
remote: E: failed to find patch for rev 4fc06fd23709e77a5ee05b1f0fde57d410fcaba6.
remote: E: failed to find patch for rev cfde74c9dbe8eed65f430334b071aeba6620882d.
remote: E: failed to find patch for rev 6e0c24e9b3f9d430dec57f61f8c12c74bca5375d.
remote: E: failed to find patch for rev 9d40291916136ed3c4480102708e75d81cc77ef1.
remote: E: failed to find patch for rev 336692f24b5ac6c39b65ff3cb971d7bd353ea4e5.
remote: E: failed to find patch for rev 58ff22a356f8003720ca9fe13749744a2bd02d98.
remote: E: failed to find patch for rev f44e54116e0c1240aa3d1f1ac0ec098a342764f5.
remote: E: failed to find patch for rev 00f04936eb6d4eb3ae9b0102f87ca3265ce80bf1.
remote: E: failed to find patch for rev a838527ccfb33716ce7ff2c4b97c0beb13d4e654.
remote: E: failed to find patch for rev a749fbc3732f469ee0ec3e36275fb7e8fe6ebd19.
remote: E: failed to find patch for rev 84597e39c734032a3b84f2ab46c35ba88b06cdf3.
remote: E: failed to find patch for rev a541eb88e4dabf506ff2b6c07e7ca19d34a316d7.
remote: E: failed to find patch for rev 4bb23bc4c444c79052ca6e79e5799d4c5db1e6da.
remote: E: failed to find patch for rev 6a8d57c0b7e243a22730f637295d620d08b975e0.
remote: E: failed to find patch for rev 39a71692eb686f6bf694969d2d068f55294846fa.
remote: E: failed to find patch for rev 13611ae96c9b2fab928eb794415f5bec6331abb2.
remote: E: failed to find patch for rev d356e03ced0873a0fe1a7a69cb05ee803e363888.
remote: E: failed to find patch for rev 1b73d2c713938f10931749ece3a703ad9004376a.
remote: E: failed to find patch for rev f5a5a84d5c551a3b1daa2c8092279fbed9abe54f.
remote: E: failed to find patch for rev 8ea8fff79c91c985884859eca49769379e06c313.
remote: E: failed to find patch for rev 3aac48c722fa836e04e95a495d758b8ee0d0cdda.
remote: E: failed to find patch for rev d83612a1e750b98b1c537fada777d615d9d6628a.
remote: E: failed to find patch for rev d9b5870edaddee69612dada2aa4bc2d8c3f70f44.
remote: I: 0 patch(es) updated to state Accepted.
remote: Sending notification emails to: openembedded-commits@lists.openembedded.org.
To git.openembedded.org:meta-openembedded
   18d518a..d9b5870  master -> master


> 
> 
> On 12/02/2016 06:29 AM, Tom Rini wrote:
> > On Fri, Dec 02, 2016 at 09:30:09AM +0100, Martin Jansa wrote:
> >> On Wed, Nov 30, 2016 at 12:41:20PM -0600, Jose Lamego wrote:
> >>>
> >>>
> >>> On 11/29/2016 07:50 PM, Khem Raj wrote:
> >>>>
> >>>>> On Nov 29, 2016, at 2:34 PM, Jose Lamego <jose.a.lamego@linux.intel.com> wrote:
> >>>>>
> >>>>>
> >>>>>
> >>>>> On 11/29/2016 02:43 PM, Tom Rini wrote:
> >>>>>> On Wed, Nov 30, 2016 at 06:52:46AM +1300, Paul Eggleton wrote:
> >>>>>>
> >>>>>>> Hi folks,
> >>>>>>>
> >>>>>>> Martin just brought to my attention that the git hook for the 
> >>>>>>> meta-openembedded repository that's supposed to update patchwork still isn't 
> >>>>>>> working and hasn't done so for a very long time. I thought we'd fixed this so 
> >>>>>>> that merged patches get marked as "Accepted", but at least for meta-
> >>>>>>> openembedded it seems like that's not the case.
> >>>>>>>
> >>>>>>> Jose / Michael - can you look into this if you haven't already? Please let me 
> >>>>>>> know if you need any help.
> >>>>>
> >>>>> The git hook was configured to update OE-core only, Michael will help us
> >>>>> to include meta-openembedded.
> >>>>>
> >>>>
> >>>> There are more repos using patchwork beyond OE-core and meta-opemebedded
> >>>> please enable it for all of them  as well.
> >>>>
> >>>
> >>> Now all of the projects are enabled to receive automatic updates from
> >>> the git-hook (Thanks Michael). Please let me know if any undesired
> >>> behavior is detected.
> >>
> >> Still doesn't mark anything as accepted:
> >>
> >> OE @ ~/meta-openembedded $ git push origin master
> >> Counting objects: 293, done.
> >> Delta compression using up to 8 threads.
> >> Compressing objects: 100% (257/257), done.
> >> Writing objects: 100% (293/293), 48.83 KiB | 0 bytes/s, done.
> >> Total 293 (delta 179), reused 68 (delta 18)
> >> remote: E: failed to find patch for rev efd3696e70a6603f1a45faa4a172433514f0a487.
> >> remote: E: failed to find patch for rev 8c720146b718aae3bc194621f3e095cf11ecb2ed.
> >> remote: E: failed to find patch for rev fc2fc9097b2fa437d4af0338e418692b4dd6c989.
> >> remote: E: failed to find patch for rev 533826b7652a2cf86ca7b665c25b98c89afc266e.
> >> remote: E: failed to find patch for rev 6afb85d29286aaa4cdd892460f6adf68912917a5.
> >> remote: E: failed to find patch for rev a7f3ec87f957ff35b8a853cd0e04c6b18bac0ece.
> >> remote: E: failed to find patch for rev 877e52fd75c7d3b88add94691df3beb5e0c54529.
> >> remote: E: failed to find patch for rev 2f6e45baba961629bfe023599144be6cfcd45728.
> >> remote: E: failed to find patch for rev 86fed80d80bc33f0a3cdbf719c00ee5f05ed4d19.
> >> remote: E: failed to find patch for rev 41d7bac5d50091cab835eac0a5eaa6c431fb2023.
> >> remote: E: failed to find patch for rev a85e12928131fce057f777daa76a7f0c75cffa14.
> >> remote: E: failed to find patch for rev 45837e6c567b1b9ff9d152a7e2a752488d313455.
> >> remote: E: failed to find patch for rev 119e75e48dbf0539b4e440417901458ffff79b38.
> >> remote: E: failed to find patch for rev a4251575f41fa5aafe5072e562497b05c11b6702.
> >> remote: E: failed to find patch for rev b6ea2c0bacb21b83bae3605ef4da080de0cad15e.
> >> remote: E: failed to find patch for rev 4580e5ae201b320ed407eae26f58655b8768b387.
> >> remote: E: failed to find patch for rev c476219e845934f767927e81fe60dbaa157df9ee.
> >> remote: E: failed to find patch for rev 0288b204f3c12af1561218fa0cfcdfb9a5508b47.
> >> remote: E: failed to find patch for rev 5596b52d64c30d8e66ec6e379cf69a9fc9407fb1.
> >> remote: E: failed to find patch for rev 24e51a6a106bc10a8838b23e4bf9ea6304314bca.
> >> remote: E: failed to find patch for rev aac6bf8bf911c3f14ff9dce08f0f011838f5a426.
> >> remote: I: 0 patch(es) updated to state Accepted.
> >> remote: Sending notification emails to: openembedded-commits@lists.openembedded.org.
> >> To git.openembedded.org:meta-openembedded
> >>    761639b..aac6bf8  master -> master
> > 
> > I don't know how far apart the freedesktop.org one is from ozlabs but
> > when I was seeing that problem locally as part of what I was working on
> > it was because of some missing host deps (that are now not required).
> > FWIW you can pretty easily turn the hook into a script to run manually
> > and sh -x it to see _why_ it's not hashing out those patches.
> > 
> 
> -- 
> Jose Lamego | OTC Embedded Platforms & Tools | GDC
> 




-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 201 bytes --]

Re: meta-openembedded patchwork git hook

[Jose Lamego]

[-- Attachment #1.1: Type: text/plain, Size: 4883 bytes --]



On 12/02/2016 09:45 AM, Jose Lamego wrote:
> I'm looking into this right now. I've filed a bug for tracking:
> https://bugzilla.yoctoproject.org/show_bug.cgi?id=10759
> 
> 
> On 12/02/2016 06:29 AM, Tom Rini wrote:
>> On Fri, Dec 02, 2016 at 09:30:09AM +0100, Martin Jansa wrote:
>>> On Wed, Nov 30, 2016 at 12:41:20PM -0600, Jose Lamego wrote:
>>>>
>>>>
>>>> On 11/29/2016 07:50 PM, Khem Raj wrote:
>>>>>
>>>>>> On Nov 29, 2016, at 2:34 PM, Jose Lamego <jose.a.lamego@linux.intel.com> wrote:
>>>>>>
>>>>>>
>>>>>>
>>>>>> On 11/29/2016 02:43 PM, Tom Rini wrote:
>>>>>>> On Wed, Nov 30, 2016 at 06:52:46AM +1300, Paul Eggleton wrote:
>>>>>>>
>>>>>>>> Hi folks,
>>>>>>>>
>>>>>>>> Martin just brought to my attention that the git hook for the 
>>>>>>>> meta-openembedded repository that's supposed to update patchwork still isn't 
>>>>>>>> working and hasn't done so for a very long time. I thought we'd fixed this so 
>>>>>>>> that merged patches get marked as "Accepted", but at least for meta-
>>>>>>>> openembedded it seems like that's not the case.
>>>>>>>>
>>>>>>>> Jose / Michael - can you look into this if you haven't already? Please let me 
>>>>>>>> know if you need any help.
>>>>>>
>>>>>> The git hook was configured to update OE-core only, Michael will help us
>>>>>> to include meta-openembedded.
>>>>>>
>>>>>
>>>>> There are more repos using patchwork beyond OE-core and meta-opemebedded
>>>>> please enable it for all of them  as well.
>>>>>
>>>>
>>>> Now all of the projects are enabled to receive automatic updates from
>>>> the git-hook (Thanks Michael). Please let me know if any undesired
>>>> behavior is detected.
>>>
>>> Still doesn't mark anything as accepted:
>>>
>>> OE @ ~/meta-openembedded $ git push origin master
>>> Counting objects: 293, done.
>>> Delta compression using up to 8 threads.
>>> Compressing objects: 100% (257/257), done.
>>> Writing objects: 100% (293/293), 48.83 KiB | 0 bytes/s, done.
>>> Total 293 (delta 179), reused 68 (delta 18)
>>> remote: E: failed to find patch for rev efd3696e70a6603f1a45faa4a172433514f0a487.
>>> remote: E: failed to find patch for rev 8c720146b718aae3bc194621f3e095cf11ecb2ed.
>>> remote: E: failed to find patch for rev fc2fc9097b2fa437d4af0338e418692b4dd6c989.
>>> remote: E: failed to find patch for rev 533826b7652a2cf86ca7b665c25b98c89afc266e.
>>> remote: E: failed to find patch for rev 6afb85d29286aaa4cdd892460f6adf68912917a5.
>>> remote: E: failed to find patch for rev a7f3ec87f957ff35b8a853cd0e04c6b18bac0ece.
>>> remote: E: failed to find patch for rev 877e52fd75c7d3b88add94691df3beb5e0c54529.
>>> remote: E: failed to find patch for rev 2f6e45baba961629bfe023599144be6cfcd45728.
>>> remote: E: failed to find patch for rev 86fed80d80bc33f0a3cdbf719c00ee5f05ed4d19.
>>> remote: E: failed to find patch for rev 41d7bac5d50091cab835eac0a5eaa6c431fb2023.
>>> remote: E: failed to find patch for rev a85e12928131fce057f777daa76a7f0c75cffa14.
>>> remote: E: failed to find patch for rev 45837e6c567b1b9ff9d152a7e2a752488d313455.
>>> remote: E: failed to find patch for rev 119e75e48dbf0539b4e440417901458ffff79b38.
>>> remote: E: failed to find patch for rev a4251575f41fa5aafe5072e562497b05c11b6702.
>>> remote: E: failed to find patch for rev b6ea2c0bacb21b83bae3605ef4da080de0cad15e.
>>> remote: E: failed to find patch for rev 4580e5ae201b320ed407eae26f58655b8768b387.
>>> remote: E: failed to find patch for rev c476219e845934f767927e81fe60dbaa157df9ee.
>>> remote: E: failed to find patch for rev 0288b204f3c12af1561218fa0cfcdfb9a5508b47.
>>> remote: E: failed to find patch for rev 5596b52d64c30d8e66ec6e379cf69a9fc9407fb1.
>>> remote: E: failed to find patch for rev 24e51a6a106bc10a8838b23e4bf9ea6304314bca.
>>> remote: E: failed to find patch for rev aac6bf8bf911c3f14ff9dce08f0f011838f5a426.
>>> remote: I: 0 patch(es) updated to state Accepted.
>>> remote: Sending notification emails to: openembedded-commits@lists.openembedded.org.
>>> To git.openembedded.org:meta-openembedded
>>>    761639b..aac6bf8  master -> master
>>
>> I don't know how far apart the freedesktop.org one is from ozlabs but
>> when I was seeing that problem locally as part of what I was working on
>> it was because of some missing host deps (that are now not required).
>> FWIW you can pretty easily turn the hook into a script to run manually
>> and sh -x it to see _why_ it's not hashing out those patches.
>>
Now all of the patchwork projects/repositories are using the
asynchronous githook that was previously only used by oe-core. This hook
version features some response time improvements (maintainers will
regain control of their machines sooner) but mainly it should solve the
automatic updating issue. We are waiting for pushes to confirm.
> 
> 
> 

-- 
Jose Lamego | OTC Embedded Platforms & Tools | GDC


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 501 bytes --]

Re: meta-openembedded patchwork git hook

[Jose Lamego]

[-- Attachment #1.1: Type: text/plain, Size: 5252 bytes --]



On 12/14/2016 02:27 PM, Jose Lamego wrote:
> 
> 
> On 12/02/2016 09:45 AM, Jose Lamego wrote:
>> I'm looking into this right now. I've filed a bug for tracking:
>> https://bugzilla.yoctoproject.org/show_bug.cgi?id=10759
>>
>>
>> On 12/02/2016 06:29 AM, Tom Rini wrote:
>>> On Fri, Dec 02, 2016 at 09:30:09AM +0100, Martin Jansa wrote:
>>>> On Wed, Nov 30, 2016 at 12:41:20PM -0600, Jose Lamego wrote:
>>>>>
>>>>>
>>>>> On 11/29/2016 07:50 PM, Khem Raj wrote:
>>>>>>
>>>>>>> On Nov 29, 2016, at 2:34 PM, Jose Lamego <jose.a.lamego@linux.intel.com> wrote:
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> On 11/29/2016 02:43 PM, Tom Rini wrote:
>>>>>>>> On Wed, Nov 30, 2016 at 06:52:46AM +1300, Paul Eggleton wrote:
>>>>>>>>
>>>>>>>>> Hi folks,
>>>>>>>>>
>>>>>>>>> Martin just brought to my attention that the git hook for the 
>>>>>>>>> meta-openembedded repository that's supposed to update patchwork still isn't 
>>>>>>>>> working and hasn't done so for a very long time. I thought we'd fixed this so 
>>>>>>>>> that merged patches get marked as "Accepted", but at least for meta-
>>>>>>>>> openembedded it seems like that's not the case.
>>>>>>>>>
>>>>>>>>> Jose / Michael - can you look into this if you haven't already? Please let me 
>>>>>>>>> know if you need any help.
>>>>>>>
>>>>>>> The git hook was configured to update OE-core only, Michael will help us
>>>>>>> to include meta-openembedded.
>>>>>>>
>>>>>>
>>>>>> There are more repos using patchwork beyond OE-core and meta-opemebedded
>>>>>> please enable it for all of them  as well.
>>>>>>
>>>>>
>>>>> Now all of the projects are enabled to receive automatic updates from
>>>>> the git-hook (Thanks Michael). Please let me know if any undesired
>>>>> behavior is detected.
>>>>
>>>> Still doesn't mark anything as accepted:
>>>>
>>>> OE @ ~/meta-openembedded $ git push origin master
>>>> Counting objects: 293, done.
>>>> Delta compression using up to 8 threads.
>>>> Compressing objects: 100% (257/257), done.
>>>> Writing objects: 100% (293/293), 48.83 KiB | 0 bytes/s, done.
>>>> Total 293 (delta 179), reused 68 (delta 18)
>>>> remote: E: failed to find patch for rev efd3696e70a6603f1a45faa4a172433514f0a487.
>>>> remote: E: failed to find patch for rev 8c720146b718aae3bc194621f3e095cf11ecb2ed.
>>>> remote: E: failed to find patch for rev fc2fc9097b2fa437d4af0338e418692b4dd6c989.
>>>> remote: E: failed to find patch for rev 533826b7652a2cf86ca7b665c25b98c89afc266e.
>>>> remote: E: failed to find patch for rev 6afb85d29286aaa4cdd892460f6adf68912917a5.
>>>> remote: E: failed to find patch for rev a7f3ec87f957ff35b8a853cd0e04c6b18bac0ece.
>>>> remote: E: failed to find patch for rev 877e52fd75c7d3b88add94691df3beb5e0c54529.
>>>> remote: E: failed to find patch for rev 2f6e45baba961629bfe023599144be6cfcd45728.
>>>> remote: E: failed to find patch for rev 86fed80d80bc33f0a3cdbf719c00ee5f05ed4d19.
>>>> remote: E: failed to find patch for rev 41d7bac5d50091cab835eac0a5eaa6c431fb2023.
>>>> remote: E: failed to find patch for rev a85e12928131fce057f777daa76a7f0c75cffa14.
>>>> remote: E: failed to find patch for rev 45837e6c567b1b9ff9d152a7e2a752488d313455.
>>>> remote: E: failed to find patch for rev 119e75e48dbf0539b4e440417901458ffff79b38.
>>>> remote: E: failed to find patch for rev a4251575f41fa5aafe5072e562497b05c11b6702.
>>>> remote: E: failed to find patch for rev b6ea2c0bacb21b83bae3605ef4da080de0cad15e.
>>>> remote: E: failed to find patch for rev 4580e5ae201b320ed407eae26f58655b8768b387.
>>>> remote: E: failed to find patch for rev c476219e845934f767927e81fe60dbaa157df9ee.
>>>> remote: E: failed to find patch for rev 0288b204f3c12af1561218fa0cfcdfb9a5508b47.
>>>> remote: E: failed to find patch for rev 5596b52d64c30d8e66ec6e379cf69a9fc9407fb1.
>>>> remote: E: failed to find patch for rev 24e51a6a106bc10a8838b23e4bf9ea6304314bca.
>>>> remote: E: failed to find patch for rev aac6bf8bf911c3f14ff9dce08f0f011838f5a426.
>>>> remote: I: 0 patch(es) updated to state Accepted.
>>>> remote: Sending notification emails to: openembedded-commits@lists.openembedded.org.
>>>> To git.openembedded.org:meta-openembedded
>>>>    761639b..aac6bf8  master -> master
>>>
>>> I don't know how far apart the freedesktop.org one is from ozlabs but
>>> when I was seeing that problem locally as part of what I was working on
>>> it was because of some missing host deps (that are now not required).
>>> FWIW you can pretty easily turn the hook into a script to run manually
>>> and sh -x it to see _why_ it's not hashing out those patches.
>>>
> Now all of the patchwork projects/repositories are using the
> asynchronous githook that was previously only used by oe-core. This hook
> version features some response time improvements (maintainers will
> regain control of their machines sooner) but mainly it should solve the
> automatic updating issue. We are waiting for pushes to confirm.

This task is being tracked in
https://bugzilla.yoctoproject.org/show_bug.cgi?id=10762

the change was implemented on Dec 13, so git pushes from this day
onwards should trigger the state update in patchwork.
>>
>>
>>
> 
> 
> 

-- 
Jose Lamego | OTC Embedded Platforms & Tools | GDC


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 501 bytes --]