← Back to team overview

curtin-dev team mailing list archive

Re: [Merge] ~nacc/curtin:nacc/lp1892494 into curtin:master

 

On Tue, Jan 26, 2021 at 7:59 PM Nish Aravamudan <nish.aravamudan@xxxxxxxxx>
wrote:

> Thanks for letting me know. I'll submit an updated branch with a testcase
> from that bug.
>

Currently the vmtest doesn't verify the apt key that's written out as
it uses add-apt-repository to add the curtin-dev repo in the
apt_config_command vmtest.  I'm not sure if add-apt-repository
uses apt-key directly or not; maybe @Paride knows; I suspect
add-apt-repository will need to change it's implementation as well
if it hasnt already.

I've a start on something that:

1) collects /etc/apt/trusted.gpg.d
2) verifies that we have the expected curtin-dev ppa .gpg file
2a) this is the binary file, ideally we'd compare it with the .asc version
      included in the config
3) add a 'filename' key to the apt source config used
4) verify that we find that file name also in /etc/apt/trusted.gpg.d/
5) collect gpg --list-keys in-target and confirm we have just one
    curtin-dev ppa key


-- 
> https://code.launchpad.net/~nacc/curtin/+git/curtin/+merge/393661
> Your team curtin developers is subscribed to branch curtin:master.
>
> --
> Mailing list: https://launchpad.net/~curtin-dev
> Post to     : curtin-dev@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~curtin-dev
> More help   : https://help.launchpad.net/ListHelp
>

https://code.launchpad.net/~nacc/curtin/+git/curtin/+merge/393661
Your team curtin developers is subscribed to branch curtin:master.


References