* Re: Patchwork not picking changes from the ML Was: [oe] [meta-python][PATCH] python-paho-mqtt: update to version 1.2
@ 2017-02-22 20:55 ` Michael Halstead
0 siblings, 0 replies; 18+ messages in thread
From: Michael Halstead @ 2017-02-22 20:55 UTC (permalink / raw)
To: Martin Jansa
Cc: Derek Straka, Lamego, Jose A, openembedded-devel,
openembedded-core
[-- Attachment #1: Type: text/plain, Size: 3152 bytes --]
I've seen several issues with hooks. I was working on them yesterday and
will continue today.
These are currently managed by hand but we are moving them into
configuration management which should help keep them working consistently.
On Feb 22, 2017 12:47 PM, "Martin Jansa" <martin.jansa@gmail.com> wrote:
> On Wed, Feb 22, 2017 at 12:45:36PM -0500, Derek Straka wrote:
> > Signed-off-by: Derek Straka <derek@asterius.io>
>
> It's not your fault Derek, but most of the patches from today and based
> on the pings from today also some earlier patches didn't make it into
> patchwork. This one is the only exception.
>
> Michael, Jose: are you aware of some issues with patchwork? Could it be
> caused by the default Reply-to change from yesterday (or day before
> during OEDAM)?
>
> I know only about issues processing merged changes:
> https://bugzilla.yoctoproject.org/show_bug.cgi?id=10967
> https://bugzilla.yoctoproject.org/show_bug.cgi?id=10759
>
> And also UI issue in patchwork that paging doesn't work, e.g.:
> https://patchwork.openembedded.org/bundle/jama/
> master-next/?submitter=&state=&q=&archive=both
> shows that there are 5 pages of changes (which should be in most cases
> marked as accepted by the hook above), but clicking on 2nd page produces
> link which drops the query parameters:
> https://patchwork.openembedded.org/bundle/jama/master-next/?page=2
> so it says "No patches to display"
>
> Regards,
>
> > ---
> > .../python/{python-paho-mqtt_1.1.bb => python-paho-mqtt_1.2.bb}
> | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> > rename meta-python/recipes-devtools/python/{python-paho-mqtt_1.1.bb =>
> python-paho-mqtt_1.2.bb} (68%)
> >
> > diff --git a/meta-python/recipes-devtools/python/python-paho-mqtt_1.1.bb
> b/meta-python/recipes-devtools/python/python-paho-mqtt_1.2.bb
> > similarity index 68%
> > rename from meta-python/recipes-devtools/python/python-paho-mqtt_1.1.bb
> > rename to meta-python/recipes-devtools/python/python-paho-mqtt_1.2.bb
> > index b8f5f3f..c8acfc5 100644
> > --- a/meta-python/recipes-devtools/python/python-paho-mqtt_1.1.bb
> > +++ b/meta-python/recipes-devtools/python/python-paho-mqtt_1.2.bb
> > @@ -11,5 +11,5 @@ RDEPENDS_${PN} = "\
> > python-threading \
> > "
> >
> > -SRC_URI[md5sum] = "a6407b74eb5e5411e157be1de5c11366"
> > -SRC_URI[sha256sum] = "0f7a629efe6e3a2c61b59d3550aa9f
> 2c4529b5689a65fde45e6f1ac36b9a261e"
> > +SRC_URI[md5sum] = "241150b3fcb920ddca4d33181f3238b1"
> > +SRC_URI[sha256sum] = "9100a6aa706ab699d414ec02705a21
> eb66f436184691d0bf1f2a85a6213c6c1f"
> > --
> > 2.7.4
> >
> > --
> > _______________________________________________
> > 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
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
>
[-- Attachment #2: Type: text/html, Size: 5677 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread* Re: [OE-core] Patchwork not picking changes from the ML Was: [meta-python][PATCH] python-paho-mqtt: update to version 1.2
2017-02-22 20:55 ` Patchwork not picking changes from the ML Was: [oe] " Michael Halstead
@ 2017-02-22 21:56 ` Jose Lamego
-1 siblings, 0 replies; 18+ messages in thread
From: Jose Lamego @ 2017-02-22 21:56 UTC (permalink / raw)
To: Michael Halstead, Martin Jansa
Cc: openembedded-core, Leonardo Sandoval, openembedded-devel,
Lamego, Jose A
[-- Attachment #1.1: Type: text/plain, Size: 4923 bytes --]
On 02/22/2017 02:55 PM, Michael Halstead wrote:
> I've seen several issues with hooks. I was working on them yesterday and
> will continue today.
>
> These are currently managed by hand but we are moving them into
> configuration management which should help keep them working consistently.
Michael: one syntax error in patchwork code was pulled into production
yesterday. This is the cause for missing patches. The error is fixed in
the Yocto repo now, please perform a server code update ASAP.
Martin: I will look at the UI issue you are describing and file a bug if
needed.
>
> On Feb 22, 2017 12:47 PM, "Martin Jansa" <martin.jansa@gmail.com
> <mailto:martin.jansa@gmail.com>> wrote:
>
> On Wed, Feb 22, 2017 at 12:45:36PM -0500, Derek Straka wrote:
> > Signed-off-by: Derek Straka <derek@asterius.io
> <mailto:derek@asterius.io>>
>
> It's not your fault Derek, but most of the patches from today and based
> on the pings from today also some earlier patches didn't make it into
> patchwork. This one is the only exception.
>
> Michael, Jose: are you aware of some issues with patchwork? Could it be
> caused by the default Reply-to change from yesterday (or day before
> during OEDAM)?
>
> I know only about issues processing merged changes:
> https://bugzilla.yoctoproject.org/show_bug.cgi?id=10967
> <https://bugzilla.yoctoproject.org/show_bug.cgi?id=10967>
> https://bugzilla.yoctoproject.org/show_bug.cgi?id=10759
> <https://bugzilla.yoctoproject.org/show_bug.cgi?id=10759>
>
> And also UI issue in patchwork that paging doesn't work, e.g.:
> https://patchwork.openembedded.org/bundle/jama/master-next/?submitter=&state=&q=&archive=both
> <https://patchwork.openembedded.org/bundle/jama/master-next/?submitter=&state=&q=&archive=both>
> shows that there are 5 pages of changes (which should be in most cases
> marked as accepted by the hook above), but clicking on 2nd page produces
> link which drops the query parameters:
> https://patchwork.openembedded.org/bundle/jama/master-next/?page=2
> <https://patchwork.openembedded.org/bundle/jama/master-next/?page=2>
> so it says "No patches to display"
>
> Regards,
>
> > ---
> > .../python/{python-paho-mqtt_1.1.bb
> <http://python-paho-mqtt_1.1.bb> => python-paho-mqtt_1.2.bb
> <http://python-paho-mqtt_1.2.bb>} | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> > rename
> meta-python/recipes-devtools/python/{python-paho-mqtt_1.1.bb
> <http://python-paho-mqtt_1.1.bb> => python-paho-mqtt_1.2.bb
> <http://python-paho-mqtt_1.2.bb>} (68%)
> >
> > diff --git
> a/meta-python/recipes-devtools/python/python-paho-mqtt_1.1.bb
> <http://python-paho-mqtt_1.1.bb>
> b/meta-python/recipes-devtools/python/python-paho-mqtt_1.2.bb
> <http://python-paho-mqtt_1.2.bb>
> > similarity index 68%
> > rename from
> meta-python/recipes-devtools/python/python-paho-mqtt_1.1.bb
> <http://python-paho-mqtt_1.1.bb>
> > rename to
> meta-python/recipes-devtools/python/python-paho-mqtt_1.2.bb
> <http://python-paho-mqtt_1.2.bb>
> > index b8f5f3f..c8acfc5 100644
> > --- a/meta-python/recipes-devtools/python/python-paho-mqtt_1.1.bb
> <http://python-paho-mqtt_1.1.bb>
> > +++ b/meta-python/recipes-devtools/python/python-paho-mqtt_1.2.bb
> <http://python-paho-mqtt_1.2.bb>
> > @@ -11,5 +11,5 @@ RDEPENDS_${PN} = "\
> > python-threading \
> > "
> >
> > -SRC_URI[md5sum] = "a6407b74eb5e5411e157be1de5c11366"
> > -SRC_URI[sha256sum] =
> "0f7a629efe6e3a2c61b59d3550aa9f2c4529b5689a65fde45e6f1ac36b9a261e"
> > +SRC_URI[md5sum] = "241150b3fcb920ddca4d33181f3238b1"
> > +SRC_URI[sha256sum] =
> "9100a6aa706ab699d414ec02705a21eb66f436184691d0bf1f2a85a6213c6c1f"
> > --
> > 2.7.4
> >
> > --
> > _______________________________________________
> > Openembedded-devel mailing list
> > Openembedded-devel@lists.openembedded.org
> <mailto:Openembedded-devel@lists.openembedded.org>
> > http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> <http://lists.openembedded.org/mailman/listinfo/openembedded-devel>
>
> --
> Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com
> <mailto:Martin.Jansa@gmail.com>
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> <mailto:Openembedded-core@lists.openembedded.org>
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
> <http://lists.openembedded.org/mailman/listinfo/openembedded-core>
>
>
>
--
Jose Lamego | OTC Embedded Platforms & Tools | GDC
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 484 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread* Re: Patchwork not picking changes from the ML Was: [oe] [meta-python][PATCH] python-paho-mqtt: update to version 1.2
@ 2017-02-22 21:56 ` Jose Lamego
0 siblings, 0 replies; 18+ messages in thread
From: Jose Lamego @ 2017-02-22 21:56 UTC (permalink / raw)
To: Michael Halstead, Martin Jansa
Cc: Derek Straka, openembedded-core, Leonardo Sandoval,
openembedded-devel, Lamego, Jose A
[-- Attachment #1.1: Type: text/plain, Size: 4923 bytes --]
On 02/22/2017 02:55 PM, Michael Halstead wrote:
> I've seen several issues with hooks. I was working on them yesterday and
> will continue today.
>
> These are currently managed by hand but we are moving them into
> configuration management which should help keep them working consistently.
Michael: one syntax error in patchwork code was pulled into production
yesterday. This is the cause for missing patches. The error is fixed in
the Yocto repo now, please perform a server code update ASAP.
Martin: I will look at the UI issue you are describing and file a bug if
needed.
>
> On Feb 22, 2017 12:47 PM, "Martin Jansa" <martin.jansa@gmail.com
> <mailto:martin.jansa@gmail.com>> wrote:
>
> On Wed, Feb 22, 2017 at 12:45:36PM -0500, Derek Straka wrote:
> > Signed-off-by: Derek Straka <derek@asterius.io
> <mailto:derek@asterius.io>>
>
> It's not your fault Derek, but most of the patches from today and based
> on the pings from today also some earlier patches didn't make it into
> patchwork. This one is the only exception.
>
> Michael, Jose: are you aware of some issues with patchwork? Could it be
> caused by the default Reply-to change from yesterday (or day before
> during OEDAM)?
>
> I know only about issues processing merged changes:
> https://bugzilla.yoctoproject.org/show_bug.cgi?id=10967
> <https://bugzilla.yoctoproject.org/show_bug.cgi?id=10967>
> https://bugzilla.yoctoproject.org/show_bug.cgi?id=10759
> <https://bugzilla.yoctoproject.org/show_bug.cgi?id=10759>
>
> And also UI issue in patchwork that paging doesn't work, e.g.:
> https://patchwork.openembedded.org/bundle/jama/master-next/?submitter=&state=&q=&archive=both
> <https://patchwork.openembedded.org/bundle/jama/master-next/?submitter=&state=&q=&archive=both>
> shows that there are 5 pages of changes (which should be in most cases
> marked as accepted by the hook above), but clicking on 2nd page produces
> link which drops the query parameters:
> https://patchwork.openembedded.org/bundle/jama/master-next/?page=2
> <https://patchwork.openembedded.org/bundle/jama/master-next/?page=2>
> so it says "No patches to display"
>
> Regards,
>
> > ---
> > .../python/{python-paho-mqtt_1.1.bb
> <http://python-paho-mqtt_1.1.bb> => python-paho-mqtt_1.2.bb
> <http://python-paho-mqtt_1.2.bb>} | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> > rename
> meta-python/recipes-devtools/python/{python-paho-mqtt_1.1.bb
> <http://python-paho-mqtt_1.1.bb> => python-paho-mqtt_1.2.bb
> <http://python-paho-mqtt_1.2.bb>} (68%)
> >
> > diff --git
> a/meta-python/recipes-devtools/python/python-paho-mqtt_1.1.bb
> <http://python-paho-mqtt_1.1.bb>
> b/meta-python/recipes-devtools/python/python-paho-mqtt_1.2.bb
> <http://python-paho-mqtt_1.2.bb>
> > similarity index 68%
> > rename from
> meta-python/recipes-devtools/python/python-paho-mqtt_1.1.bb
> <http://python-paho-mqtt_1.1.bb>
> > rename to
> meta-python/recipes-devtools/python/python-paho-mqtt_1.2.bb
> <http://python-paho-mqtt_1.2.bb>
> > index b8f5f3f..c8acfc5 100644
> > --- a/meta-python/recipes-devtools/python/python-paho-mqtt_1.1.bb
> <http://python-paho-mqtt_1.1.bb>
> > +++ b/meta-python/recipes-devtools/python/python-paho-mqtt_1.2.bb
> <http://python-paho-mqtt_1.2.bb>
> > @@ -11,5 +11,5 @@ RDEPENDS_${PN} = "\
> > python-threading \
> > "
> >
> > -SRC_URI[md5sum] = "a6407b74eb5e5411e157be1de5c11366"
> > -SRC_URI[sha256sum] =
> "0f7a629efe6e3a2c61b59d3550aa9f2c4529b5689a65fde45e6f1ac36b9a261e"
> > +SRC_URI[md5sum] = "241150b3fcb920ddca4d33181f3238b1"
> > +SRC_URI[sha256sum] =
> "9100a6aa706ab699d414ec02705a21eb66f436184691d0bf1f2a85a6213c6c1f"
> > --
> > 2.7.4
> >
> > --
> > _______________________________________________
> > Openembedded-devel mailing list
> > Openembedded-devel@lists.openembedded.org
> <mailto:Openembedded-devel@lists.openembedded.org>
> > http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> <http://lists.openembedded.org/mailman/listinfo/openembedded-devel>
>
> --
> Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com
> <mailto:Martin.Jansa@gmail.com>
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> <mailto:Openembedded-core@lists.openembedded.org>
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
> <http://lists.openembedded.org/mailman/listinfo/openembedded-core>
>
>
>
--
Jose Lamego | OTC Embedded Platforms & Tools | GDC
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 484 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread* Re: [OE-core] Patchwork not picking changes from the ML
2017-02-22 21:56 ` Patchwork not picking changes from the ML Was: [oe] " Jose Lamego
@ 2017-02-24 14:21 ` Patrick Ohly
-1 siblings, 0 replies; 18+ messages in thread
From: Patrick Ohly @ 2017-02-24 14:21 UTC (permalink / raw)
To: Jose Lamego
Cc: openembedded-core, Lamego, Jose A, openembedded-devel,
Leonardo Sandoval
On Wed, 2017-02-22 at 15:56 -0600, Jose Lamego wrote:
>
> On 02/22/2017 02:55 PM, Michael Halstead wrote:
> > I've seen several issues with hooks. I was working on them yesterday and
> > will continue today.
> >
> > These are currently managed by hand but we are moving them into
> > configuration management which should help keep them working consistently.
>
> Michael: one syntax error in patchwork code was pulled into production
> yesterday. This is the cause for missing patches. The error is fixed in
> the Yocto repo now, please perform a server code update ASAP.
>
> Martin: I will look at the UI issue you are describing and file a bug if
> needed.
Would it perhaps make sense to reply to the original author with an
email confirming that his patch is now in Patchwork? It should include a
link to the patch series, too.
This could have several advantages:
* submitters not aware of Patchwork or whether their target
currently uses it learn about it and then can follow the
progress of their patch
* everyone gets a confirmation that the submission made it through
the various mail servers and Patchwork itself
It still relies on the original submitter to watch out for breakages in
the processes, but I guess that can't be avoided with an asynchronous,
mail-based process.
--
Best Regards, Patrick Ohly
The content of this message is my personal opinion only and although
I am an employee of Intel, the statements I make here in no way
represent Intel's position on the issue, nor am I authorized to speak
on behalf of Intel on this matter.
^ permalink raw reply [flat|nested] 18+ messages in thread* Re: Patchwork not picking changes from the ML
@ 2017-02-24 14:21 ` Patrick Ohly
0 siblings, 0 replies; 18+ messages in thread
From: Patrick Ohly @ 2017-02-24 14:21 UTC (permalink / raw)
To: Jose Lamego
Cc: Derek Straka, openembedded-core, Lamego, Jose A,
openembedded-devel, Leonardo Sandoval
On Wed, 2017-02-22 at 15:56 -0600, Jose Lamego wrote:
>
> On 02/22/2017 02:55 PM, Michael Halstead wrote:
> > I've seen several issues with hooks. I was working on them yesterday and
> > will continue today.
> >
> > These are currently managed by hand but we are moving them into
> > configuration management which should help keep them working consistently.
>
> Michael: one syntax error in patchwork code was pulled into production
> yesterday. This is the cause for missing patches. The error is fixed in
> the Yocto repo now, please perform a server code update ASAP.
>
> Martin: I will look at the UI issue you are describing and file a bug if
> needed.
Would it perhaps make sense to reply to the original author with an
email confirming that his patch is now in Patchwork? It should include a
link to the patch series, too.
This could have several advantages:
* submitters not aware of Patchwork or whether their target
currently uses it learn about it and then can follow the
progress of their patch
* everyone gets a confirmation that the submission made it through
the various mail servers and Patchwork itself
It still relies on the original submitter to watch out for breakages in
the processes, but I guess that can't be avoided with an asynchronous,
mail-based process.
--
Best Regards, Patrick Ohly
The content of this message is my personal opinion only and although
I am an employee of Intel, the statements I make here in no way
represent Intel's position on the issue, nor am I authorized to speak
on behalf of Intel on this matter.
^ permalink raw reply [flat|nested] 18+ messages in thread* Re: Patchwork not picking changes from the ML
2017-02-24 14:21 ` Patrick Ohly
(?)
@ 2017-02-24 14:33 ` Gary Thomas
2017-02-24 16:27 ` Patrick Ohly
-1 siblings, 1 reply; 18+ messages in thread
From: Gary Thomas @ 2017-02-24 14:33 UTC (permalink / raw)
To: openembedded-core
On 2017-02-24 15:21, Patrick Ohly wrote:
> On Wed, 2017-02-22 at 15:56 -0600, Jose Lamego wrote:
>>
>> On 02/22/2017 02:55 PM, Michael Halstead wrote:
>>> I've seen several issues with hooks. I was working on them yesterday and
>>> will continue today.
>>>
>>> These are currently managed by hand but we are moving them into
>>> configuration management which should help keep them working consistently.
>>
>> Michael: one syntax error in patchwork code was pulled into production
>> yesterday. This is the cause for missing patches. The error is fixed in
>> the Yocto repo now, please perform a server code update ASAP.
>>
>> Martin: I will look at the UI issue you are describing and file a bug if
>> needed.
>
> Would it perhaps make sense to reply to the original author with an
> email confirming that his patch is now in Patchwork? It should include a
> link to the patch series, too.
>
> This could have several advantages:
> * submitters not aware of Patchwork or whether their target
> currently uses it learn about it and then can follow the
> progress of their patch
> * everyone gets a confirmation that the submission made it through
> the various mail servers and Patchwork itself
>
> It still relies on the original submitter to watch out for breakages in
> the processes, but I guess that can't be avoided with an asynchronous,
> mail-based process.
>
I would love to see this added to the process - +1 :-)
--
------------------------------------------------------------
Gary Thomas | Consulting for the
MLB Associates | Embedded world
------------------------------------------------------------
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: Patchwork not picking changes from the ML
2017-02-24 14:33 ` Gary Thomas
@ 2017-02-24 16:27 ` Patrick Ohly
2017-02-24 16:46 ` Burton, Ross
2017-02-24 16:50 ` Jose Lamego
0 siblings, 2 replies; 18+ messages in thread
From: Patrick Ohly @ 2017-02-24 16:27 UTC (permalink / raw)
To: Gary Thomas; +Cc: openembedded-core
On Fri, 2017-02-24 at 15:33 +0100, Gary Thomas wrote:
> On 2017-02-24 15:21, Patrick Ohly wrote:
> > On Wed, 2017-02-22 at 15:56 -0600, Jose Lamego wrote:
> >>
> >> On 02/22/2017 02:55 PM, Michael Halstead wrote:
> >>> I've seen several issues with hooks. I was working on them yesterday and
> >>> will continue today.
> >>>
> >>> These are currently managed by hand but we are moving them into
> >>> configuration management which should help keep them working consistently.
> >>
> >> Michael: one syntax error in patchwork code was pulled into production
> >> yesterday. This is the cause for missing patches. The error is fixed in
> >> the Yocto repo now, please perform a server code update ASAP.
> >>
> >> Martin: I will look at the UI issue you are describing and file a bug if
> >> needed.
> >
> > Would it perhaps make sense to reply to the original author with an
> > email confirming that his patch is now in Patchwork? It should include a
> > link to the patch series, too.
> >
> > This could have several advantages:
> > * submitters not aware of Patchwork or whether their target
> > currently uses it learn about it and then can follow the
> > progress of their patch
> > * everyone gets a confirmation that the submission made it through
> > the various mail servers and Patchwork itself
> >
> > It still relies on the original submitter to watch out for breakages in
> > the processes, but I guess that can't be avoided with an asynchronous,
> > mail-based process.
> >
>
> I would love to see this added to the process - +1 :-)
Let me clarify that my original proposal was to reply only to the
original author. That was meant to keep noise down on the list. However,
perhaps it should also go to the list?
Then others can help check that Patchwork works, as the original author
might not be aware that a response is missing. It also tells everyone
the relevant link in Patchwork.
--
Best Regards, Patrick Ohly
The content of this message is my personal opinion only and although
I am an employee of Intel, the statements I make here in no way
represent Intel's position on the issue, nor am I authorized to speak
on behalf of Intel on this matter.
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: Patchwork not picking changes from the ML
2017-02-24 16:27 ` Patrick Ohly
@ 2017-02-24 16:46 ` Burton, Ross
2017-02-25 4:29 ` Gary Thomas
2017-02-24 16:50 ` Jose Lamego
1 sibling, 1 reply; 18+ messages in thread
From: Burton, Ross @ 2017-02-24 16:46 UTC (permalink / raw)
To: Patrick Ohly; +Cc: Gary Thomas, OE-core
[-- Attachment #1: Type: text/plain, Size: 544 bytes --]
On 24 February 2017 at 16:27, Patrick Ohly <patrick.ohly@intel.com> wrote:
> Let me clarify that my original proposal was to reply only to the
> original author. That was meant to keep noise down on the list. However,
> perhaps it should also go to the list?
>
> Then others can help check that Patchwork works, as the original author
> might not be aware that a response is missing. It also tells everyone
> the relevant link in Patchwork.
>
As long as patchworks ignores the email it sends, that's not a terrible
idea.
Ross
[-- Attachment #2: Type: text/html, Size: 965 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: Patchwork not picking changes from the ML
2017-02-24 16:46 ` Burton, Ross
@ 2017-02-25 4:29 ` Gary Thomas
0 siblings, 0 replies; 18+ messages in thread
From: Gary Thomas @ 2017-02-25 4:29 UTC (permalink / raw)
To: openembedded-core
On 2017-02-24 17:46, Burton, Ross wrote:
>
> On 24 February 2017 at 16:27, Patrick Ohly <patrick.ohly@intel.com <mailto:patrick.ohly@intel.com>> wrote:
>
> Let me clarify that my original proposal was to reply only to the
> original author. That was meant to keep noise down on the list. However,
> perhaps it should also go to the list?
>
> Then others can help check that Patchwork works, as the original author
> might not be aware that a response is missing. It also tells everyone
> the relevant link in Patchwork.
>
>
> As long as patchworks ignores the email it sends, that's not a terrible idea.
I'd be happy either way, as long as I (as the poster) get some
sort of feedback that the patch has been received and queued.
Thanks for improving this system :-)
--
------------------------------------------------------------
Gary Thomas | Consulting for the
MLB Associates | Embedded world
------------------------------------------------------------
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: Patchwork not picking changes from the ML
2017-02-24 16:27 ` Patrick Ohly
2017-02-24 16:46 ` Burton, Ross
@ 2017-02-24 16:50 ` Jose Lamego
1 sibling, 0 replies; 18+ messages in thread
From: Jose Lamego @ 2017-02-24 16:50 UTC (permalink / raw)
To: openembedded-core
[-- Attachment #1.1: Type: text/plain, Size: 2718 bytes --]
On 02/24/2017 10:27 AM, Patrick Ohly wrote:
> On Fri, 2017-02-24 at 15:33 +0100, Gary Thomas wrote:
>> On 2017-02-24 15:21, Patrick Ohly wrote:
>>> On Wed, 2017-02-22 at 15:56 -0600, Jose Lamego wrote:
>>>>
>>>> On 02/22/2017 02:55 PM, Michael Halstead wrote:
>>>>> I've seen several issues with hooks. I was working on them yesterday and
>>>>> will continue today.
>>>>>
>>>>> These are currently managed by hand but we are moving them into
>>>>> configuration management which should help keep them working consistently.
>>>>
>>>> Michael: one syntax error in patchwork code was pulled into production
>>>> yesterday. This is the cause for missing patches. The error is fixed in
>>>> the Yocto repo now, please perform a server code update ASAP.
>>>>
>>>> Martin: I will look at the UI issue you are describing and file a bug if
>>>> needed.
>>>
>>> Would it perhaps make sense to reply to the original author with an
>>> email confirming that his patch is now in Patchwork? It should include a
>>> link to the patch series, too.
>>>
>>> This could have several advantages:
>>> * submitters not aware of Patchwork or whether their target
>>> currently uses it learn about it and then can follow the
>>> progress of their patch
>>> * everyone gets a confirmation that the submission made it through
>>> the various mail servers and Patchwork itself
>>>
>>> It still relies on the original submitter to watch out for breakages in
>>> the processes, but I guess that can't be avoided with an asynchronous,
>>> mail-based process.
>>>
>>
>> I would love to see this added to the process - +1 :-)
>
> Let me clarify that my original proposal was to reply only to the
> original author. That was meant to keep noise down on the list. However,
> perhaps it should also go to the list?
>
> Then others can help check that Patchwork works, as the original author
> might not be aware that a response is missing. It also tells everyone
> the relevant link in Patchwork.
>
There is a filed request to enable email notifications in patchwork:
https://bugzilla.yoctoproject.org/show_bug.cgi?id=7684
this notifications will let users (both submitter and CC'ed-users) know
through an email about any patch status change (including "New"). Users
will be able to opt-out of such notifications.
However, this do not covers the case when submitter is not registered as
a Patchwork user. So, there might be a one-time-only email notification
to this kind of submitter, as Patrick suggests.
If I don't get any objections for this, I'll file a request in bugzilla
next week.
--
Jose Lamego | OTC Embedded Platforms & Tools | GDC
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 484 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [OE-core] Patchwork not picking changes from the ML Was: [meta-python][PATCH] python-paho-mqtt: update to version 1.2
2017-02-22 20:55 ` Patchwork not picking changes from the ML Was: [oe] " Michael Halstead
@ 2017-02-22 21:56 ` Derek Straka
-1 siblings, 0 replies; 18+ messages in thread
From: Derek Straka @ 2017-02-22 21:56 UTC (permalink / raw)
To: Michael Halstead; +Cc: openembedded-core, Lamego, Jose A, openembeded-devel
Martin - No problem. I'll resend the patches.
Michael - Would it be worth waiting before I resend the change sets? It
looks like at least a subset of them have made it through.
Thanks.
-Derek
On Wed, Feb 22, 2017 at 3:55 PM, Michael Halstead <
mhalstead@linuxfoundation.org> wrote:
> I've seen several issues with hooks. I was working on them yesterday and
> will continue today.
>
> These are currently managed by hand but we are moving them into
> configuration management which should help keep them working consistently.
>
> On Feb 22, 2017 12:47 PM, "Martin Jansa" <martin.jansa@gmail.com> wrote:
>
>> On Wed, Feb 22, 2017 at 12:45:36PM -0500, Derek Straka wrote:
>> > Signed-off-by: Derek Straka <derek@asterius.io>
>>
>> It's not your fault Derek, but most of the patches from today and based
>> on the pings from today also some earlier patches didn't make it into
>> patchwork. This one is the only exception.
>>
>> Michael, Jose: are you aware of some issues with patchwork? Could it be
>> caused by the default Reply-to change from yesterday (or day before
>> during OEDAM)?
>>
>> I know only about issues processing merged changes:
>> https://bugzilla.yoctoproject.org/show_bug.cgi?id=10967
>> https://bugzilla.yoctoproject.org/show_bug.cgi?id=10759
>>
>> And also UI issue in patchwork that paging doesn't work, e.g.:
>> https://patchwork.openembedded.org/bundle/jama/master-next/?
>> submitter=&state=&q=&archive=both
>> shows that there are 5 pages of changes (which should be in most cases
>> marked as accepted by the hook above), but clicking on 2nd page produces
>> link which drops the query parameters:
>> https://patchwork.openembedded.org/bundle/jama/master-next/?page=2
>> so it says "No patches to display"
>>
>> Regards,
>>
>> > ---
>> > .../python/{python-paho-mqtt_1.1.bb => python-paho-mqtt_1.2.bb}
>> | 4 ++--
>> > 1 file changed, 2 insertions(+), 2 deletions(-)
>> > rename meta-python/recipes-devtools/python/{python-paho-mqtt_1.1.bb
>> => python-paho-mqtt_1.2.bb} (68%)
>> >
>> > diff --git a/meta-python/recipes-devtools/python/python-paho-mqtt_1.1.
>> bb b/meta-python/recipes-devtools/python/python-paho-mqtt_1.2.bb
>> > similarity index 68%
>> > rename from meta-python/recipes-devtools/python/python-paho-mqtt_1.1.bb
>> > rename to meta-python/recipes-devtools/python/python-paho-mqtt_1.2.bb
>> > index b8f5f3f..c8acfc5 100644
>> > --- a/meta-python/recipes-devtools/python/python-paho-mqtt_1.1.bb
>> > +++ b/meta-python/recipes-devtools/python/python-paho-mqtt_1.2.bb
>> > @@ -11,5 +11,5 @@ RDEPENDS_${PN} = "\
>> > python-threading \
>> > "
>> >
>> > -SRC_URI[md5sum] = "a6407b74eb5e5411e157be1de5c11366"
>> > -SRC_URI[sha256sum] = "0f7a629efe6e3a2c61b59d3550aa9
>> f2c4529b5689a65fde45e6f1ac36b9a261e"
>> > +SRC_URI[md5sum] = "241150b3fcb920ddca4d33181f3238b1"
>> > +SRC_URI[sha256sum] = "9100a6aa706ab699d414ec02705a2
>> 1eb66f436184691d0bf1f2a85a6213c6c1f"
>> > --
>> > 2.7.4
>> >
>> > --
>> > _______________________________________________
>> > 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
>>
>> --
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>>
>>
^ permalink raw reply [flat|nested] 18+ messages in thread* Re: Patchwork not picking changes from the ML Was: [oe] [meta-python][PATCH] python-paho-mqtt: update to version 1.2
@ 2017-02-22 21:56 ` Derek Straka
0 siblings, 0 replies; 18+ messages in thread
From: Derek Straka @ 2017-02-22 21:56 UTC (permalink / raw)
To: Michael Halstead; +Cc: openembedded-core, Lamego, Jose A, openembeded-devel
[-- Attachment #1: Type: text/plain, Size: 3537 bytes --]
Martin - No problem. I'll resend the patches.
Michael - Would it be worth waiting before I resend the change sets? It
looks like at least a subset of them have made it through.
Thanks.
-Derek
On Wed, Feb 22, 2017 at 3:55 PM, Michael Halstead <
mhalstead@linuxfoundation.org> wrote:
> I've seen several issues with hooks. I was working on them yesterday and
> will continue today.
>
> These are currently managed by hand but we are moving them into
> configuration management which should help keep them working consistently.
>
> On Feb 22, 2017 12:47 PM, "Martin Jansa" <martin.jansa@gmail.com> wrote:
>
>> On Wed, Feb 22, 2017 at 12:45:36PM -0500, Derek Straka wrote:
>> > Signed-off-by: Derek Straka <derek@asterius.io>
>>
>> It's not your fault Derek, but most of the patches from today and based
>> on the pings from today also some earlier patches didn't make it into
>> patchwork. This one is the only exception.
>>
>> Michael, Jose: are you aware of some issues with patchwork? Could it be
>> caused by the default Reply-to change from yesterday (or day before
>> during OEDAM)?
>>
>> I know only about issues processing merged changes:
>> https://bugzilla.yoctoproject.org/show_bug.cgi?id=10967
>> https://bugzilla.yoctoproject.org/show_bug.cgi?id=10759
>>
>> And also UI issue in patchwork that paging doesn't work, e.g.:
>> https://patchwork.openembedded.org/bundle/jama/master-next/?
>> submitter=&state=&q=&archive=both
>> shows that there are 5 pages of changes (which should be in most cases
>> marked as accepted by the hook above), but clicking on 2nd page produces
>> link which drops the query parameters:
>> https://patchwork.openembedded.org/bundle/jama/master-next/?page=2
>> so it says "No patches to display"
>>
>> Regards,
>>
>> > ---
>> > .../python/{python-paho-mqtt_1.1.bb => python-paho-mqtt_1.2.bb}
>> | 4 ++--
>> > 1 file changed, 2 insertions(+), 2 deletions(-)
>> > rename meta-python/recipes-devtools/python/{python-paho-mqtt_1.1.bb
>> => python-paho-mqtt_1.2.bb} (68%)
>> >
>> > diff --git a/meta-python/recipes-devtools/python/python-paho-mqtt_1.1.
>> bb b/meta-python/recipes-devtools/python/python-paho-mqtt_1.2.bb
>> > similarity index 68%
>> > rename from meta-python/recipes-devtools/python/python-paho-mqtt_1.1.bb
>> > rename to meta-python/recipes-devtools/python/python-paho-mqtt_1.2.bb
>> > index b8f5f3f..c8acfc5 100644
>> > --- a/meta-python/recipes-devtools/python/python-paho-mqtt_1.1.bb
>> > +++ b/meta-python/recipes-devtools/python/python-paho-mqtt_1.2.bb
>> > @@ -11,5 +11,5 @@ RDEPENDS_${PN} = "\
>> > python-threading \
>> > "
>> >
>> > -SRC_URI[md5sum] = "a6407b74eb5e5411e157be1de5c11366"
>> > -SRC_URI[sha256sum] = "0f7a629efe6e3a2c61b59d3550aa9
>> f2c4529b5689a65fde45e6f1ac36b9a261e"
>> > +SRC_URI[md5sum] = "241150b3fcb920ddca4d33181f3238b1"
>> > +SRC_URI[sha256sum] = "9100a6aa706ab699d414ec02705a2
>> 1eb66f436184691d0bf1f2a85a6213c6c1f"
>> > --
>> > 2.7.4
>> >
>> > --
>> > _______________________________________________
>> > 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
>>
>> --
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>>
>>
[-- Attachment #2: Type: text/html, Size: 6525 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread* Re: [OE-core] Patchwork not picking changes from the ML Was: [meta-python][PATCH] python-paho-mqtt: update to version 1.2
2017-02-22 21:56 ` Patchwork not picking changes from the ML Was: [oe] " Derek Straka
@ 2017-02-22 22:08 ` Michael Halstead
-1 siblings, 0 replies; 18+ messages in thread
From: Michael Halstead @ 2017-02-22 22:08 UTC (permalink / raw)
To: Derek Straka; +Cc: openembeded-devel, openembedded-core, Lamego, Jose A
On Feb 22, 2017 1:56 PM, "Derek Straka" <derek@asterius.io> wrote:
Martin - No problem. I'll resend the patches.
Michael - Would it be worth waiting before I resend the change sets? It
looks like at least a subset of them have made it through.
Please wait while I make sure this problem is correctly resolved before
re-sending. I'll e-mail once we're ready.
Thanks.
-Derek
On Wed, Feb 22, 2017 at 3:55 PM, Michael Halstead <
mhalstead@linuxfoundation.org> wrote:
> I've seen several issues with hooks. I was working on them yesterday and
> will continue today.
>
> These are currently managed by hand but we are moving them into
> configuration management which should help keep them working consistently.
>
> On Feb 22, 2017 12:47 PM, "Martin Jansa" <martin.jansa@gmail.com> wrote:
>
>> On Wed, Feb 22, 2017 at 12:45:36PM -0500, Derek Straka wrote:
>> > Signed-off-by: Derek Straka <derek@asterius.io>
>>
>> It's not your fault Derek, but most of the patches from today and based
>> on the pings from today also some earlier patches didn't make it into
>> patchwork. This one is the only exception.
>>
>> Michael, Jose: are you aware of some issues with patchwork? Could it be
>> caused by the default Reply-to change from yesterday (or day before
>> during OEDAM)?
>>
>> I know only about issues processing merged changes:
>> https://bugzilla.yoctoproject.org/show_bug.cgi?id=10967
>> https://bugzilla.yoctoproject.org/show_bug.cgi?id=10759
>>
>> And also UI issue in patchwork that paging doesn't work, e.g.:
>> https://patchwork.openembedded.org/bundle/jama/master-next/?
>> submitter=&state=&q=&archive=both
>> shows that there are 5 pages of changes (which should be in most cases
>> marked as accepted by the hook above), but clicking on 2nd page produces
>> link which drops the query parameters:
>> https://patchwork.openembedded.org/bundle/jama/master-next/?page=2
>> so it says "No patches to display"
>>
>> Regards,
>>
>> > ---
>> > .../python/{python-paho-mqtt_1.1.bb => python-paho-mqtt_1.2.bb}
>> | 4 ++--
>> > 1 file changed, 2 insertions(+), 2 deletions(-)
>> > rename meta-python/recipes-devtools/python/{python-paho-mqtt_1.1.bb
>> => python-paho-mqtt_1.2.bb} (68%)
>> >
>> > diff --git a/meta-python/recipes-devtools/python/python-paho-mqtt_1.1.b
>> b b/meta-python/recipes-devtools/python/python-paho-mqtt_1.2.bb
>> > similarity index 68%
>> > rename from meta-python/recipes-devtools/python/python-paho-mqtt_1.1.bb
>> > rename to meta-python/recipes-devtools/python/python-paho-mqtt_1.2.bb
>> > index b8f5f3f..c8acfc5 100644
>> > --- a/meta-python/recipes-devtools/python/python-paho-mqtt_1.1.bb
>> > +++ b/meta-python/recipes-devtools/python/python-paho-mqtt_1.2.bb
>> > @@ -11,5 +11,5 @@ RDEPENDS_${PN} = "\
>> > python-threading \
>> > "
>> >
>> > -SRC_URI[md5sum] = "a6407b74eb5e5411e157be1de5c11366"
>> > -SRC_URI[sha256sum] = "0f7a629efe6e3a2c61b59d3550aa9
>> f2c4529b5689a65fde45e6f1ac36b9a261e"
>> > +SRC_URI[md5sum] = "241150b3fcb920ddca4d33181f3238b1"
>> > +SRC_URI[sha256sum] = "9100a6aa706ab699d414ec02705a2
>> 1eb66f436184691d0bf1f2a85a6213c6c1f"
>> > --
>> > 2.7.4
>> >
>> > --
>> > _______________________________________________
>> > 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
>>
>> --
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>>
>>
> Michael: one syntax error in patchwork code was pulled into production
> yesterday. This is the cause for missing patches. The error is fixed in
> the Yocto repo now, please perform a server code update ASAP.
Leo and I pulled changes and restarted so this should be in place now.
^ permalink raw reply [flat|nested] 18+ messages in thread* Re: Patchwork not picking changes from the ML Was: [oe] [meta-python][PATCH] python-paho-mqtt: update to version 1.2
@ 2017-02-22 22:08 ` Michael Halstead
0 siblings, 0 replies; 18+ messages in thread
From: Michael Halstead @ 2017-02-22 22:08 UTC (permalink / raw)
To: Derek Straka; +Cc: openembeded-devel, openembedded-core, Lamego, Jose A
[-- Attachment #1: Type: text/plain, Size: 4021 bytes --]
On Feb 22, 2017 1:56 PM, "Derek Straka" <derek@asterius.io> wrote:
Martin - No problem. I'll resend the patches.
Michael - Would it be worth waiting before I resend the change sets? It
looks like at least a subset of them have made it through.
Please wait while I make sure this problem is correctly resolved before
re-sending. I'll e-mail once we're ready.
Thanks.
-Derek
On Wed, Feb 22, 2017 at 3:55 PM, Michael Halstead <
mhalstead@linuxfoundation.org> wrote:
> I've seen several issues with hooks. I was working on them yesterday and
> will continue today.
>
> These are currently managed by hand but we are moving them into
> configuration management which should help keep them working consistently.
>
> On Feb 22, 2017 12:47 PM, "Martin Jansa" <martin.jansa@gmail.com> wrote:
>
>> On Wed, Feb 22, 2017 at 12:45:36PM -0500, Derek Straka wrote:
>> > Signed-off-by: Derek Straka <derek@asterius.io>
>>
>> It's not your fault Derek, but most of the patches from today and based
>> on the pings from today also some earlier patches didn't make it into
>> patchwork. This one is the only exception.
>>
>> Michael, Jose: are you aware of some issues with patchwork? Could it be
>> caused by the default Reply-to change from yesterday (or day before
>> during OEDAM)?
>>
>> I know only about issues processing merged changes:
>> https://bugzilla.yoctoproject.org/show_bug.cgi?id=10967
>> https://bugzilla.yoctoproject.org/show_bug.cgi?id=10759
>>
>> And also UI issue in patchwork that paging doesn't work, e.g.:
>> https://patchwork.openembedded.org/bundle/jama/master-next/?
>> submitter=&state=&q=&archive=both
>> shows that there are 5 pages of changes (which should be in most cases
>> marked as accepted by the hook above), but clicking on 2nd page produces
>> link which drops the query parameters:
>> https://patchwork.openembedded.org/bundle/jama/master-next/?page=2
>> so it says "No patches to display"
>>
>> Regards,
>>
>> > ---
>> > .../python/{python-paho-mqtt_1.1.bb => python-paho-mqtt_1.2.bb}
>> | 4 ++--
>> > 1 file changed, 2 insertions(+), 2 deletions(-)
>> > rename meta-python/recipes-devtools/python/{python-paho-mqtt_1.1.bb
>> => python-paho-mqtt_1.2.bb} (68%)
>> >
>> > diff --git a/meta-python/recipes-devtools/python/python-paho-mqtt_1.1.b
>> b b/meta-python/recipes-devtools/python/python-paho-mqtt_1.2.bb
>> > similarity index 68%
>> > rename from meta-python/recipes-devtools/python/python-paho-mqtt_1.1.bb
>> > rename to meta-python/recipes-devtools/python/python-paho-mqtt_1.2.bb
>> > index b8f5f3f..c8acfc5 100644
>> > --- a/meta-python/recipes-devtools/python/python-paho-mqtt_1.1.bb
>> > +++ b/meta-python/recipes-devtools/python/python-paho-mqtt_1.2.bb
>> > @@ -11,5 +11,5 @@ RDEPENDS_${PN} = "\
>> > python-threading \
>> > "
>> >
>> > -SRC_URI[md5sum] = "a6407b74eb5e5411e157be1de5c11366"
>> > -SRC_URI[sha256sum] = "0f7a629efe6e3a2c61b59d3550aa9
>> f2c4529b5689a65fde45e6f1ac36b9a261e"
>> > +SRC_URI[md5sum] = "241150b3fcb920ddca4d33181f3238b1"
>> > +SRC_URI[sha256sum] = "9100a6aa706ab699d414ec02705a2
>> 1eb66f436184691d0bf1f2a85a6213c6c1f"
>> > --
>> > 2.7.4
>> >
>> > --
>> > _______________________________________________
>> > 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
>>
>> --
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>>
>>
> Michael: one syntax error in patchwork code was pulled into production
> yesterday. This is the cause for missing patches. The error is fixed in
> the Yocto repo now, please perform a server code update ASAP.
Leo and I pulled changes and restarted so this should be in place now.
[-- Attachment #2: Type: text/html, Size: 8293 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread