Diff comments:

> diff --git a/charms/focal/autopkgtest-web/webcontrol/browse.cgi 
> b/charms/focal/autopkgtest-web/webcontrol/browse.cgi
> index f2dc00f..0f7ebe1 100755
> --- a/charms/focal/autopkgtest-web/webcontrol/browse.cgi
> +++ b/charms/focal/autopkgtest-web/webcontrol/browse.cgi
> @@ -25,6 +25,8 @@ SUPPORTED_UBUNTU_RELEASES = sorted(
>      set(UDI.supported() + UDI.supported_esm()), key=ALL_UBUNTU_RELEASES.index
>  )
>  
> +INDEXED_PACKAGES_FP = "/home/ubuntu/indexed-packages.json"

Maybe this should go into a config file?

> +
>  
>  def init_config():
>      global db_con, swift_container_url


-- 
https://code.launchpad.net/~andersson123/autopkgtest-cloud/+git/autopkgtest-cloud/+merge/458001
Your team Canonical's Ubuntu QA is requested to review the proposed merge of 
~andersson123/autopkgtest-cloud:improve-loading-results into 
autopkgtest-cloud:master.


-- 
Mailing list: https://launchpad.net/~canonical-ubuntu-qa
Post to     : canonical-ubuntu-qa@lists.launchpad.net
Unsubscribe : https://launchpad.net/~canonical-ubuntu-qa
More help   : https://help.launchpad.net/ListHelp

Reply via email to