mbox series

[0/4] apt configuration fixes

Message ID 20220205151528.9199-1-henning.schild@siemens.com
Headers show
Series apt configuration fixes | expand

Message

Henning Schild Feb. 5, 2022, 5:15 a.m. UTC
This attempts to solve the apt fetch retry problem in a better way. On
the way it was found that bootstrap leaves a file around, which probably
should not remain in images.

Henning Schild (4):
  rootfs: configure apt to always retry downloads
  Revert "deb-dl-dir: Retry downloads 3 times for src files"
  Revert "rootfs: Retry downloads 3 times"
  bootstrap: do not leave 50isar.conf in final rootfss

 meta/classes/deb-dl-dir.bbclass                |  2 +-
 meta/classes/image.bbclass                     |  1 +
 meta/classes/rootfs.bbclass                    | 18 +++++++++++++++++-
 .../isar-bootstrap/files/isar-apt.conf         |  2 --
 .../isar-bootstrap/isar-bootstrap.inc          |  5 +----
 5 files changed, 20 insertions(+), 8 deletions(-)
 delete mode 100644 meta/recipes-core/isar-bootstrap/files/isar-apt.conf

Comments

Henning Schild Feb. 5, 2022, 5:18 a.m. UTC | #1
I could not really test this because it needs a shaky network. Cutting
network will be detected as such and retries would not happen anyways.
At least the ways i tried. But i see no reason for this to not work.

regards,
Henning

Am Sat,  5 Feb 2022 16:15:24 +0100
schrieb Henning Schild <henning.schild@siemens.com>:

> This attempts to solve the apt fetch retry problem in a better way. On
> the way it was found that bootstrap leaves a file around, which
> probably should not remain in images.
> 
> Henning Schild (4):
>   rootfs: configure apt to always retry downloads
>   Revert "deb-dl-dir: Retry downloads 3 times for src files"
>   Revert "rootfs: Retry downloads 3 times"
>   bootstrap: do not leave 50isar.conf in final rootfss
> 
>  meta/classes/deb-dl-dir.bbclass                |  2 +-
>  meta/classes/image.bbclass                     |  1 +
>  meta/classes/rootfs.bbclass                    | 18
> +++++++++++++++++- .../isar-bootstrap/files/isar-apt.conf         |
> 2 -- .../isar-bootstrap/isar-bootstrap.inc          |  5 +----
>  5 files changed, 20 insertions(+), 8 deletions(-)
>  delete mode 100644
> meta/recipes-core/isar-bootstrap/files/isar-apt.conf
>
Anton Mikanovich Feb. 20, 2022, 11:24 p.m. UTC | #2
5.02.22 18:15, henning.schild@siemens.com wrote:
> This attempts to solve the apt fetch retry problem in a better way. On
> the way it was found that bootstrap leaves a file around, which probably
> should not remain in images.
>
> Henning Schild (4):
>    rootfs: configure apt to always retry downloads
>    Revert "deb-dl-dir: Retry downloads 3 times for src files"
>    Revert "rootfs: Retry downloads 3 times"
>    bootstrap: do not leave 50isar.conf in final rootfss
>
>   meta/classes/deb-dl-dir.bbclass                |  2 +-
>   meta/classes/image.bbclass                     |  1 +
>   meta/classes/rootfs.bbclass                    | 18 +++++++++++++++++-
>   .../isar-bootstrap/files/isar-apt.conf         |  2 --
>   .../isar-bootstrap/isar-bootstrap.inc          |  5 +----
>   5 files changed, 20 insertions(+), 8 deletions(-)
>   delete mode 100644 meta/recipes-core/isar-bootstrap/files/isar-apt.conf
>
Applied to next, thanks.
Henning Schild Feb. 21, 2022, 6:12 a.m. UTC | #3
Am Mon, 21 Feb 2022 12:24:09 +0300
schrieb Anton Mikanovich <amikan@ilbers.de>:

> 5.02.22 18:15, henning.schild@siemens.com wrote:
> > This attempts to solve the apt fetch retry problem in a better way.
> > On the way it was found that bootstrap leaves a file around, which
> > probably should not remain in images.
> >
> > Henning Schild (4):
> >    rootfs: configure apt to always retry downloads
> >    Revert "deb-dl-dir: Retry downloads 3 times for src files"
> >    Revert "rootfs: Retry downloads 3 times"
> >    bootstrap: do not leave 50isar.conf in final rootfss
> >
> >   meta/classes/deb-dl-dir.bbclass                |  2 +-
> >   meta/classes/image.bbclass                     |  1 +
> >   meta/classes/rootfs.bbclass                    | 18
> > +++++++++++++++++- .../isar-bootstrap/files/isar-apt.conf         |
> >  2 -- .../isar-bootstrap/isar-bootstrap.inc          |  5 +----
> >   5 files changed, 20 insertions(+), 8 deletions(-)
> >   delete mode 100644
> > meta/recipes-core/isar-bootstrap/files/isar-apt.conf 
> Applied to next, thanks.
> 

Thanks! I see only now that the "From:" in my emails was messed with by
the SMTP server i am now forced to use. Any chance we can force-push
next to fix that?

What i did send and wanted was 
"Henning Schild <henning.schild@siemens.com>" but what came out is
"henning.schild@siemens.com <henning.schild@siemens.com>"

Please let me know if a force-push is possible, i would prepare a
"next" on github for a hard reset.

regards,
Henning
Baurzhan Ismagulov Feb. 21, 2022, 6:51 a.m. UTC | #4
On Mon, Feb 21, 2022 at 05:12:55PM +0100, Henning Schild wrote:
> Thanks! I see only now that the "From:" in my emails was messed with by
> the SMTP server i am now forced to use. Any chance we can force-push
> next to fix that?
> 
> What i did send and wanted was 
> "Henning Schild <henning.schild@siemens.com>" but what came out is
> "henning.schild@siemens.com <henning.schild@siemens.com>"
> 
> Please let me know if a force-push is possible, i would prepare a
> "next" on github for a hard reset.

Fixed, please check.

In general, I'd like to avoid rewriting history because it's, well, history :)
.

With kind regards,
Baurzhan.
Henning Schild Feb. 21, 2022, 8:24 a.m. UTC | #5
Am Mon, 21 Feb 2022 17:51:05 +0100
schrieb Baurzhan Ismagulov <ibr@radix50.net>:

> On Mon, Feb 21, 2022 at 05:12:55PM +0100, Henning Schild wrote:
> > Thanks! I see only now that the "From:" in my emails was messed
> > with by the SMTP server i am now forced to use. Any chance we can
> > force-push next to fix that?
> > 
> > What i did send and wanted was 
> > "Henning Schild <henning.schild@siemens.com>" but what came out is
> > "henning.schild@siemens.com <henning.schild@siemens.com>"
> > 
> > Please let me know if a force-push is possible, i would prepare a
> > "next" on github for a hard reset.  
> 
> Fixed, please check.
> 
> In general, I'd like to avoid rewriting history because it's, well,
> history :) .

Thanks a million! I know that history rewriting is bad and isar usually
does not do it ... even on "next" where other projects might be more
relaxed about it.

I sure hope it does not happen again and it would be asking too much
for the maintainer to maybe double-check. But if anyone sees a "weird"
"From" just speak up and ask the contributor to check their mail setup.

I personally would do my best to deliver patches again and try to help
people to fix such issues (especially those trying to send From
@siemens.com). 

regards,
Henning

> With kind regards,
> Baurzhan.
>
Jan Kiszka Feb. 21, 2022, 9:20 a.m. UTC | #6
On 21.02.22 17:12, Henning Schild wrote:
> Am Mon, 21 Feb 2022 12:24:09 +0300
> schrieb Anton Mikanovich <amikan@ilbers.de>:
> 
>> 5.02.22 18:15, henning.schild@siemens.com wrote:
>>> This attempts to solve the apt fetch retry problem in a better way.
>>> On the way it was found that bootstrap leaves a file around, which
>>> probably should not remain in images.
>>>
>>> Henning Schild (4):
>>>    rootfs: configure apt to always retry downloads
>>>    Revert "deb-dl-dir: Retry downloads 3 times for src files"
>>>    Revert "rootfs: Retry downloads 3 times"
>>>    bootstrap: do not leave 50isar.conf in final rootfss
>>>
>>>   meta/classes/deb-dl-dir.bbclass                |  2 +-
>>>   meta/classes/image.bbclass                     |  1 +
>>>   meta/classes/rootfs.bbclass                    | 18
>>> +++++++++++++++++- .../isar-bootstrap/files/isar-apt.conf         |
>>>  2 -- .../isar-bootstrap/isar-bootstrap.inc          |  5 +----
>>>   5 files changed, 20 insertions(+), 8 deletions(-)
>>>   delete mode 100644
>>> meta/recipes-core/isar-bootstrap/files/isar-apt.conf 
>> Applied to next, thanks.
>>
> 
> Thanks! I see only now that the "From:" in my emails was messed with by
> the SMTP server i am now forced to use. Any chance we can force-push
> next to fix that?
> 
> What i did send and wanted was 
> "Henning Schild <henning.schild@siemens.com>" but what came out is
> "henning.schild@siemens.com <henning.schild@siemens.com>"
> 

I heard these complaints from other as well. Since our servers started
rewriting but also because of me sometime using different servers, I
*always* force git to inject a From: line into the commit messages. That
makes you independent of those manglings.

BTW, my local maintainer tool tries to detect such cases, at least when
there is also a signed-off line, and that one differs from the author's
line.

Jan
Henning Schild Feb. 21, 2022, 10:30 a.m. UTC | #7
Am Mon, 21 Feb 2022 20:20:21 +0100
schrieb Jan Kiszka <jan.kiszka@siemens.com>:

> On 21.02.22 17:12, Henning Schild wrote:
> > Am Mon, 21 Feb 2022 12:24:09 +0300
> > schrieb Anton Mikanovich <amikan@ilbers.de>:
> >   
> >> 5.02.22 18:15, henning.schild@siemens.com wrote:  
> >>> This attempts to solve the apt fetch retry problem in a better
> >>> way. On the way it was found that bootstrap leaves a file around,
> >>> which probably should not remain in images.
> >>>
> >>> Henning Schild (4):
> >>>    rootfs: configure apt to always retry downloads
> >>>    Revert "deb-dl-dir: Retry downloads 3 times for src files"
> >>>    Revert "rootfs: Retry downloads 3 times"
> >>>    bootstrap: do not leave 50isar.conf in final rootfss
> >>>
> >>>   meta/classes/deb-dl-dir.bbclass                |  2 +-
> >>>   meta/classes/image.bbclass                     |  1 +
> >>>   meta/classes/rootfs.bbclass                    | 18
> >>> +++++++++++++++++- .../isar-bootstrap/files/isar-apt.conf
> >>> | 2 -- .../isar-bootstrap/isar-bootstrap.inc          |  5 +----
> >>>   5 files changed, 20 insertions(+), 8 deletions(-)
> >>>   delete mode 100644
> >>> meta/recipes-core/isar-bootstrap/files/isar-apt.conf   
> >> Applied to next, thanks.
> >>  
> > 
> > Thanks! I see only now that the "From:" in my emails was messed
> > with by the SMTP server i am now forced to use. Any chance we can
> > force-push next to fix that?
> > 
> > What i did send and wanted was 
> > "Henning Schild <henning.schild@siemens.com>" but what came out is
> > "henning.schild@siemens.com <henning.schild@siemens.com>"
> >   
> 
> I heard these complaints from other as well. Since our servers started
> rewriting but also because of me sometime using different servers, I
> *always* force git to inject a From: line into the commit messages.
> That makes you independent of those manglings.

How do you force git? I am using my own force since a couple of years
... not sure why it failed this time.

https://github.com/henning-schild/git/commit/ca44f026a31537477e66559f0bf0f3acd2deecdf

I should probably go ahead and mainline that or something similar.

> BTW, my local maintainer tool tries to detect such cases, at least
> when there is also a signed-off line, and that one differs from the
> author's line.

Maybe such a tool should not remain "local", but really it should not
be required if mail servers would simply send what they are told to.

In fact i will likely switch to sending from "@hennsch.de" soon, not
caring anymore.

regards,
Henning

> Jan
>
Florian Bezdeka Feb. 21, 2022, 11:01 p.m. UTC | #8
On Mon, 2022-02-21 at 21:30 +0100, Henning Schild wrote:
> Am Mon, 21 Feb 2022 20:20:21 +0100
> schrieb Jan Kiszka <jan.kiszka@siemens.com>:
> 
> > On 21.02.22 17:12, Henning Schild wrote:
> > > Am Mon, 21 Feb 2022 12:24:09 +0300
> > > schrieb Anton Mikanovich <amikan@ilbers.de>:
> > >   
> > > > 5.02.22 18:15, henning.schild@siemens.com wrote:  
> > > > > This attempts to solve the apt fetch retry problem in a better
> > > > > way. On the way it was found that bootstrap leaves a file around,
> > > > > which probably should not remain in images.
> > > > > 
> > > > > Henning Schild (4):
> > > > >    rootfs: configure apt to always retry downloads
> > > > >    Revert "deb-dl-dir: Retry downloads 3 times for src files"
> > > > >    Revert "rootfs: Retry downloads 3 times"
> > > > >    bootstrap: do not leave 50isar.conf in final rootfss
> > > > > 
> > > > >   meta/classes/deb-dl-dir.bbclass                |  2 +-
> > > > >   meta/classes/image.bbclass                     |  1 +
> > > > >   meta/classes/rootfs.bbclass                    | 18
> > > > > +++++++++++++++++- .../isar-bootstrap/files/isar-apt.conf
> > > > > > 2 -- .../isar-bootstrap/isar-bootstrap.inc          |  5 +----
> > > > >   5 files changed, 20 insertions(+), 8 deletions(-)
> > > > >   delete mode 100644
> > > > > meta/recipes-core/isar-bootstrap/files/isar-apt.conf   
> > > > Applied to next, thanks.
> > > >  
> > > 
> > > Thanks! I see only now that the "From:" in my emails was messed
> > > with by the SMTP server i am now forced to use. Any chance we can
> > > force-push next to fix that?
> > > 
> > > What i did send and wanted was 
> > > "Henning Schild <henning.schild@siemens.com>" but what came out is
> > > "henning.schild@siemens.com <henning.schild@siemens.com>"
> > >   
> > 
> > I heard these complaints from other as well. Since our servers started
> > rewriting but also because of me sometime using different servers, I
> > *always* force git to inject a From: line into the commit messages.
> > That makes you independent of those manglings.
> 
> How do you force git? I am using my own force since a couple of years
> ... not sure why it failed this time.

I'm forcing git send-email to repeat the FROM header inside the body by
setting sendemail.from in my ~/.gitconfig to a value that is different
from my "git commit identity" (user.name and user.email):

[sendemail]
    from = F. Bezdeka <email>

> 
> https://github.com/henning-schild/git/commit/ca44f026a31537477e66559f0bf0f3acd2deecdf
> 
> I should probably go ahead and mainline that or something similar.
> 
> > BTW, my local maintainer tool tries to detect such cases, at least
> > when there is also a signed-off line, and that one differs from the
> > author's line.
> 
> Maybe such a tool should not remain "local", but really it should not
> be required if mail servers would simply send what they are told to.
> 
> In fact i will likely switch to sending from "@hennsch.de" soon, not
> caring anymore.
> 
> regards,
> Henning
> 
> > Jan
> > 
>
Jan Kiszka Feb. 21, 2022, 11:06 p.m. UTC | #9
On 21.02.22 21:30, Henning Schild wrote:
> Am Mon, 21 Feb 2022 20:20:21 +0100
> schrieb Jan Kiszka <jan.kiszka@siemens.com>:
> 
>> On 21.02.22 17:12, Henning Schild wrote:
>>> Am Mon, 21 Feb 2022 12:24:09 +0300
>>> schrieb Anton Mikanovich <amikan@ilbers.de>:
>>>   
>>>> 5.02.22 18:15, henning.schild@siemens.com wrote:  
>>>>> This attempts to solve the apt fetch retry problem in a better
>>>>> way. On the way it was found that bootstrap leaves a file around,
>>>>> which probably should not remain in images.
>>>>>
>>>>> Henning Schild (4):
>>>>>    rootfs: configure apt to always retry downloads
>>>>>    Revert "deb-dl-dir: Retry downloads 3 times for src files"
>>>>>    Revert "rootfs: Retry downloads 3 times"
>>>>>    bootstrap: do not leave 50isar.conf in final rootfss
>>>>>
>>>>>   meta/classes/deb-dl-dir.bbclass                |  2 +-
>>>>>   meta/classes/image.bbclass                     |  1 +
>>>>>   meta/classes/rootfs.bbclass                    | 18
>>>>> +++++++++++++++++- .../isar-bootstrap/files/isar-apt.conf
>>>>> | 2 -- .../isar-bootstrap/isar-bootstrap.inc          |  5 +----
>>>>>   5 files changed, 20 insertions(+), 8 deletions(-)
>>>>>   delete mode 100644
>>>>> meta/recipes-core/isar-bootstrap/files/isar-apt.conf   
>>>> Applied to next, thanks.
>>>>  
>>>
>>> Thanks! I see only now that the "From:" in my emails was messed
>>> with by the SMTP server i am now forced to use. Any chance we can
>>> force-push next to fix that?
>>>
>>> What i did send and wanted was 
>>> "Henning Schild <henning.schild@siemens.com>" but what came out is
>>> "henning.schild@siemens.com <henning.schild@siemens.com>"
>>>   
>>
>> I heard these complaints from other as well. Since our servers started
>> rewriting but also because of me sometime using different servers, I
>> *always* force git to inject a From: line into the commit messages.
>> That makes you independent of those manglings.
> 
> How do you force git? I am using my own force since a couple of years
> ... not sure why it failed this time.
> 
> https://github.com/henning-schild/git/commit/ca44f026a31537477e66559f0bf0f3acd2deecdf
> 
> I should probably go ahead and mainline that or something similar.

git format-patch --form="bla"
  -> adds "From:" to body (unless you committed as "bla")
git send-email --from="real@user.com"

> 
>> BTW, my local maintainer tool tries to detect such cases, at least
>> when there is also a signed-off line, and that one differs from the
>> author's line.
> 
> Maybe such a tool should not remain "local", but really it should not
> be required if mail servers would simply send what they are told to.

https://addons.thunderbird.net/de/thunderbird/addon/copy-patch/

Jan

> 
> In fact i will likely switch to sending from "@hennsch.de" soon, not
> caring anymore.
> 
> regards,
> Henning
> 
>> Jan
>>
>