[Sts-sponsors] [Merge] ~cgrabowski/maas-ci/+git/maas-ci-internal:fix_terraform_cleanup into ~maas-committers/maas-ci/+git/maas-ci-internal:main

2022-11-29 Thread mp+433758
The proposal to merge 
~cgrabowski/maas-ci/+git/maas-ci-internal:fix_terraform_cleanup into 
~maas-committers/maas-ci/+git/maas-ci-internal:main has been updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~cgrabowski/maas-ci/+git/maas-ci-internal/+merge/433758
-- 
Your team MAAS Committers is subscribed to branch 
~maas-committers/maas-ci/+git/maas-ci-internal:main.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


[Sts-sponsors] [Merge] ~cgrabowski/maas-ci/+git/maas-ci-internal:fix_terraform_cleanup into ~maas-committers/maas-ci/+git/maas-ci-internal:main

2022-11-29 Thread Christian Grabowski
The proposal to merge 
~cgrabowski/maas-ci/+git/maas-ci-internal:fix_terraform_cleanup into 
~maas-committers/maas-ci/+git/maas-ci-internal:main has been updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~cgrabowski/maas-ci/+git/maas-ci-internal/+merge/433758
-- 
Your team MAAS Committers is subscribed to branch 
~maas-committers/maas-ci/+git/maas-ci-internal:main.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


Re: [Sts-sponsors] [Merge] ~cgrabowski/maas-ci/+git/maas-ci-internal:fix_terraform_cleanup into ~maas-committers/maas-ci/+git/maas-ci-internal:main

2022-11-29 Thread Diego Mascialino
Review: Approve

+1
-- 
https://code.launchpad.net/~cgrabowski/maas-ci/+git/maas-ci-internal/+merge/433758
Your team MAAS Committers is subscribed to branch 
~maas-committers/maas-ci/+git/maas-ci-internal:main.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


[Sts-sponsors] [Merge] ~bjornt/maas:test_handles_missing_system_handler_on_notification-debug into maas:master

2022-11-29 Thread MAAS Lander
The proposal to merge 
~bjornt/maas:test_handles_missing_system_handler_on_notification-debug into 
maas:master has been updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~bjornt/maas/+git/maas/+merge/433741
-- 
Your team MAAS Committers is subscribed to branch maas:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


[Sts-sponsors] [Merge] ~bjornt/maas:pytest-django-setup into maas:master

2022-11-29 Thread MAAS Lander
The proposal to merge ~bjornt/maas:pytest-django-setup into maas:master has 
been updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~bjornt/maas/+git/maas/+merge/433712
-- 
Your team MAAS Committers is subscribed to branch maas:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


[Sts-sponsors] [Merge] ~bjornt/maas:test_handles_missing_system_handler_on_notification-debug into maas:master

2022-11-29 Thread Björn Tillenius
The proposal to merge 
~bjornt/maas:test_handles_missing_system_handler_on_notification-debug into 
maas:master has been updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~bjornt/maas/+git/maas/+merge/433741
-- 
Your team MAAS Committers is subscribed to branch maas:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


Re: [Sts-sponsors] [Merge] ~bjornt/maas:test_handles_missing_system_handler_on_notification-debug into maas:master

2022-11-29 Thread Christian Grabowski
Review: Approve

+1
-- 
https://code.launchpad.net/~bjornt/maas/+git/maas/+merge/433741
Your team MAAS Committers is subscribed to branch maas:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


[Sts-sponsors] [Merge] ~bjornt/maas:pytest-django-setup into maas:master

2022-11-29 Thread Björn Tillenius
The proposal to merge ~bjornt/maas:pytest-django-setup into maas:master has 
been updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~bjornt/maas/+git/maas/+merge/433712
-- 
Your team MAAS Committers is subscribed to branch maas:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


[Sts-sponsors] [Merge] ~bjornt/maas-ci/+git/maas-ci-internal:pytest-no-external-db into ~maas-committers/maas-ci/+git/maas-ci-internal:main

2022-11-29 Thread mp+433713
The proposal to merge 
~bjornt/maas-ci/+git/maas-ci-internal:pytest-no-external-db into 
~maas-committers/maas-ci/+git/maas-ci-internal:main has been updated.

Status: Needs review => Merged

For more details, see:
https://code.launchpad.net/~bjornt/maas-ci/+git/maas-ci-internal/+merge/433713
-- 
Your team MAAS Committers is subscribed to branch 
~maas-committers/maas-ci/+git/maas-ci-internal:main.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


[Sts-sponsors] [Merge] ~dmascialino/maas-ci/+git/system-tests:update_o11y_setup into ~maas-committers/maas-ci/+git/system-tests:master

2022-11-29 Thread MAAS Lander
The proposal to merge ~dmascialino/maas-ci/+git/system-tests:update_o11y_setup 
into ~maas-committers/maas-ci/+git/system-tests:master has been updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~dmascialino/maas-ci/+git/system-tests/+merge/433801
-- 
Your team MAAS Committers is subscribed to branch 
~maas-committers/maas-ci/+git/system-tests:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


[Sts-sponsors] [Merge] ~dmascialino/maas-ci/+git/system-tests:update_o11y_setup into ~maas-committers/maas-ci/+git/system-tests:master

2022-11-29 Thread Diego Mascialino
The proposal to merge ~dmascialino/maas-ci/+git/system-tests:update_o11y_setup 
into ~maas-committers/maas-ci/+git/system-tests:master has been updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~dmascialino/maas-ci/+git/system-tests/+merge/433801
-- 
Your team MAAS Committers is subscribed to branch 
~maas-committers/maas-ci/+git/system-tests:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


Re: [Sts-sponsors] [UNITTESTS] -b update_o11y_setup lp:~dmascialino/maas-ci/+git/system-tests into -b master lp:~maas-committers/maas-ci/+git/system-tests - TESTS PASS

2022-11-29 Thread MAAS Lander
Review: Approve

UNIT TESTS
-b update_o11y_setup lp:~dmascialino/maas-ci/+git/system-tests into -b master 
lp:~maas-committers/maas-ci/+git/system-tests

STATUS: SUCCESS
COMMIT: 4a484869c6fc2cbb3be31cd09ce98da2f5c08415
-- 
https://code.launchpad.net/~dmascialino/maas-ci/+git/system-tests/+merge/433801
Your team MAAS Committers is subscribed to branch 
~maas-committers/maas-ci/+git/system-tests:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


Re: [Sts-sponsors] [Merge] ~dmascialino/maas-ci/+git/system-tests:update_o11y_setup into ~maas-committers/maas-ci/+git/system-tests:master

2022-11-29 Thread Alexsander de Souza
Review: Approve

+1
-- 
https://code.launchpad.net/~dmascialino/maas-ci/+git/system-tests/+merge/433801
Your team MAAS Committers is subscribed to branch 
~maas-committers/maas-ci/+git/system-tests:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


[Sts-sponsors] [Merge] ~dmascialino/maas-ci/+git/system-tests:allow_list_of_ppa into ~maas-committers/maas-ci/+git/system-tests:master

2022-11-29 Thread MAAS Lander
The proposal to merge ~dmascialino/maas-ci/+git/system-tests:allow_list_of_ppa 
into ~maas-committers/maas-ci/+git/system-tests:master has been updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~dmascialino/maas-ci/+git/system-tests/+merge/433798
-- 
Your team MAAS Committers is subscribed to branch 
~maas-committers/maas-ci/+git/system-tests:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


[Sts-sponsors] [Merge] ~dmascialino/maas-ci/+git/system-tests:update_o11y_setup into ~maas-committers/maas-ci/+git/system-tests:master

2022-11-29 Thread Diego Mascialino
Diego Mascialino has proposed merging 
~dmascialino/maas-ci/+git/system-tests:update_o11y_setup into 
~maas-committers/maas-ci/+git/system-tests:master.

Commit message:
Update o11y systemd unit.

Requested reviews:
  MAAS Committers (maas-committers)

For more details, see:
https://code.launchpad.net/~dmascialino/maas-ci/+git/system-tests/+merge/433801
-- 
Your team MAAS Committers is requested to review the proposed merge of 
~dmascialino/maas-ci/+git/system-tests:update_o11y_setup into 
~maas-committers/maas-ci/+git/system-tests:master.
diff --git a/systemtests/fixtures.py b/systemtests/fixtures.py
index 033fa08..4e5610c 100644
--- a/systemtests/fixtures.py
+++ b/systemtests/fixtures.py
@@ -129,6 +129,9 @@ def maas_deb_repo(
 build_container, ["git", "-C", "maas", "checkout", maas_git_branch]
 )
 lxd.execute(
+build_container, ["git", "config", "--global", "core.abbrev", "9"]
+)
+lxd.execute(
 build_container,
 [
 "mk-build-deps",
@@ -573,7 +576,8 @@ def maas_region(
 -E MAAS_AZ="default" \
 {AGENT_PATH} \
 -config.expand-env \
--config.file=/opt/agent/agent.yml
+-config.file=/opt/agent/agent.yml \
+-server.http.address="0.0.0.0:3100" -server.grpc.address="0.0.0.0:9095"
 """
 )
 lxd.execute(maas_container, ["sh", "-c", telemetry_run_cmd])
-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


[Sts-sponsors] [Merge] ~dmascialino/maas-ci/+git/system-tests:allow_list_of_ppa into ~maas-committers/maas-ci/+git/system-tests:master

2022-11-29 Thread Diego Mascialino
The proposal to merge ~dmascialino/maas-ci/+git/system-tests:allow_list_of_ppa 
into ~maas-committers/maas-ci/+git/system-tests:master has been updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~dmascialino/maas-ci/+git/system-tests/+merge/433798
-- 
Your team MAAS Committers is subscribed to branch 
~maas-committers/maas-ci/+git/system-tests:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


Re: [Sts-sponsors] [Merge] ~dmascialino/maas-ci/+git/system-tests:allow_list_of_ppa into ~maas-committers/maas-ci/+git/system-tests:master

2022-11-29 Thread Alexsander de Souza
Review: Approve

+1
-- 
https://code.launchpad.net/~dmascialino/maas-ci/+git/system-tests/+merge/433798
Your team MAAS Committers is subscribed to branch 
~maas-committers/maas-ci/+git/system-tests:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


[Sts-sponsors] [Merge] ~cgrabowski/maas:fix_dns_restart_when_adding_vault into maas:master

2022-11-29 Thread MAAS Lander
The proposal to merge ~cgrabowski/maas:fix_dns_restart_when_adding_vault into 
maas:master has been updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~cgrabowski/maas/+git/maas/+merge/433738
-- 
Your team MAAS Committers is subscribed to branch maas:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


Re: [Sts-sponsors] [UNITTESTS] -b allow_list_of_ppa lp:~dmascialino/maas-ci/+git/system-tests into -b master lp:~maas-committers/maas-ci/+git/system-tests - TESTS PASS

2022-11-29 Thread MAAS Lander
Review: Approve

UNIT TESTS
-b allow_list_of_ppa lp:~dmascialino/maas-ci/+git/system-tests into -b master 
lp:~maas-committers/maas-ci/+git/system-tests

STATUS: SUCCESS
COMMIT: 4219d7c51b464ca99fd7bd4a55c6ecb9d6c49937
-- 
https://code.launchpad.net/~dmascialino/maas-ci/+git/system-tests/+merge/433798
Your team MAAS Committers is requested to review the proposed merge of 
~dmascialino/maas-ci/+git/system-tests:allow_list_of_ppa into 
~maas-committers/maas-ci/+git/system-tests:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


[Sts-sponsors] [Merge] ~dmascialino/maas-ci/+git/system-tests:allow_list_of_ppa into ~maas-committers/maas-ci/+git/system-tests:master

2022-11-29 Thread Diego Mascialino
Diego Mascialino has proposed merging 
~dmascialino/maas-ci/+git/system-tests:allow_list_of_ppa into 
~maas-committers/maas-ci/+git/system-tests:master.

Commit message:
Allow list of PPAs



Requested reviews:
  MAAS Committers (maas-committers)

For more details, see:
https://code.launchpad.net/~dmascialino/maas-ci/+git/system-tests/+merge/433798

Our utils/gen_config.py generates a list of PPAs.

This MP fixes system-tests code to use it correctly.
-- 
Your team MAAS Committers is requested to review the proposed merge of 
~dmascialino/maas-ci/+git/system-tests:allow_list_of_ppa into 
~maas-committers/maas-ci/+git/system-tests:master.
diff --git a/config.yaml.sample b/config.yaml.sample
index 991b6cb..e9648fe 100644
--- a/config.yaml.sample
+++ b/config.yaml.sample
@@ -87,7 +87,8 @@ machines:
 -END PRIVATE KEY-
 
 deb:
-ppa: ppa:maas-committers/latest-deps
+ppa:
+- ppa:maas-committers/latest-deps
 git_repo: https://git.launchpad.net/maas
 git_branch: master
 
diff --git a/systemtests/fixtures.py b/systemtests/fixtures.py
index 033fa08..63e9baf 100644
--- a/systemtests/fixtures.py
+++ b/systemtests/fixtures.py
@@ -28,12 +28,13 @@ LXD_PROFILE = os.environ.get("MAAS_SYSTEMTESTS_LXD_PROFILE", "prof-maas-lab")
 
 def _add_maas_ppa(lxd: CLILXD, container: str, config: dict[str, Any]) -> None:
 """Add MAAS PPA to the given container."""
-MAAS_PPA = config.get("deb", {}).get("ppa", "ppa:maas-committers/latest-deps")
-lxd.execute(
-container,
-["add-apt-repository", "-y", MAAS_PPA],
-environment={"DEBIAN_FRONTEND": "noninteractive"},
-)
+MAAS_PPA = config.get("deb", {}).get("ppa", ["ppa:maas-committers/latest-deps"])
+for ppa in MAAS_PPA:
+lxd.execute(
+container,
+["add-apt-repository", "-y", ppa],
+environment={"DEBIAN_FRONTEND": "noninteractive"},
+)
 
 
 @pytest.fixture(scope="session")
-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


[Sts-sponsors] [Merge] ~dmascialino/maas-ci/+git/system-tests:allow_list_of_ppa into maas-ci:maas-submodules-sync

2022-11-29 Thread Diego Mascialino
The proposal to merge ~dmascialino/maas-ci/+git/system-tests:allow_list_of_ppa 
into maas-ci:maas-submodules-sync has been updated.

Status: Needs review => Rejected

For more details, see:
https://code.launchpad.net/~dmascialino/maas-ci/+git/system-tests/+merge/433752
-- 
Your team MAAS Committers is requested to review the proposed merge of 
~dmascialino/maas-ci/+git/system-tests:allow_list_of_ppa into 
maas-ci:maas-submodules-sync.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


[Sts-sponsors] [Merge] ~cgrabowski/maas:fix_dns_restart_when_adding_vault into maas:master

2022-11-29 Thread Christian Grabowski
The proposal to merge ~cgrabowski/maas:fix_dns_restart_when_adding_vault into 
maas:master has been updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~cgrabowski/maas/+git/maas/+merge/433738
-- 
Your team MAAS Committers is subscribed to branch maas:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


[Sts-sponsors] [Merge] ~alexsander-souza/maas:lp1997975_fix_grafana_agent_conf into maas:master

2022-11-29 Thread MAAS Lander
The proposal to merge ~alexsander-souza/maas:lp1997975_fix_grafana_agent_conf 
into maas:master has been updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~alexsander-souza/maas/+git/maas/+merge/433674
-- 
Your team MAAS Committers is subscribed to branch maas:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


Re: [Sts-sponsors] [Merge] ~bjornt/maas:pytest-django-setup into maas:master

2022-11-29 Thread Alexsander de Souza
Review: Approve

+1
-- 
https://code.launchpad.net/~bjornt/maas/+git/maas/+merge/433712
Your team MAAS Committers is subscribed to branch maas:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


Re: [Sts-sponsors] [Merge] ~bjornt/maas-ci/+git/maas-ci-internal:pytest-no-external-db into ~maas-committers/maas-ci/+git/maas-ci-internal:main

2022-11-29 Thread Alexsander de Souza
Review: Approve

+1
-- 
https://code.launchpad.net/~bjornt/maas-ci/+git/maas-ci-internal/+merge/433713
Your team MAAS Committers is subscribed to branch 
~maas-committers/maas-ci/+git/maas-ci-internal:main.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


Re: [Sts-sponsors] [Merge] ~cgrabowski/maas:fix_dns_restart_when_adding_vault into maas:master

2022-11-29 Thread Alexsander de Souza
Review: Approve

+1
-- 
https://code.launchpad.net/~cgrabowski/maas/+git/maas/+merge/433738
Your team MAAS Committers is subscribed to branch maas:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


[Sts-sponsors] [Merge] ~alexsander-souza/maas/+git/maas-release-tools:remove_dup_check into ~maas-committers/maas/+git/maas-release-tools:main

2022-11-29 Thread mp+433615
The proposal to merge 
~alexsander-souza/maas/+git/maas-release-tools:remove_dup_check into 
~maas-committers/maas/+git/maas-release-tools:main has been updated.

Status: Needs review => Merged

For more details, see:
https://code.launchpad.net/~alexsander-souza/maas/+git/maas-release-tools/+merge/433615
-- 
Your team MAAS Committers is subscribed to branch 
~maas-committers/maas/+git/maas-release-tools:main.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


[Sts-sponsors] [Merge] ~alexsander-souza/maas:lp1997975_fix_grafana_agent_conf into maas:master

2022-11-29 Thread Alexsander de Souza
The proposal to merge ~alexsander-souza/maas:lp1997975_fix_grafana_agent_conf 
into maas:master has been updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~alexsander-souza/maas/+git/maas/+merge/433674
-- 
Your team MAAS Committers is subscribed to branch maas:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


Re: [Sts-sponsors] [UNITTESTS] -b vault-refactor-cache lp:~igor-brovtsin/maas/+git/maas into -b master lp:~maas-committers/maas - TESTS FAILED

2022-11-29 Thread MAAS Lander
Review: Needs Fixing

UNIT TESTS
-b vault-refactor-cache lp:~igor-brovtsin/maas/+git/maas into -b master 
lp:~maas-committers/maas

STATUS: FAILED
LOG: http://maas-ci.internal:8080/job/maas-tester/1528/consoleText
COMMIT: 554ae68aaed6560f04675559fc8ee88bcef766f0
-- 
https://code.launchpad.net/~igor-brovtsin/maas/+git/maas/+merge/433663
Your team MAAS Maintainers is requested to review the proposed merge of 
~igor-brovtsin/maas:vault-refactor-cache into maas:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


Re: [Sts-sponsors] [Merge] ~igor-brovtsin/maas:vault-refactor-cache into maas:master

2022-11-29 Thread Igor Brovtsin
jenkins: !test
-- 
https://code.launchpad.net/~igor-brovtsin/maas/+git/maas/+merge/433663
Your team MAAS Maintainers is requested to review the proposed merge of 
~igor-brovtsin/maas:vault-refactor-cache into maas:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


[Sts-sponsors] [Bug 1989446] Re: [SRU] unable to boot guest with large memory when SEV is enabled on host

2022-11-29 Thread Julian Andres Klode
I have no plans for individual Backports as we plan to copy back 2.06
after the security update is out.

The security update is currently blocked by getting the 2022v1 signing
key installed in a PPA and having the previous security update copied
out of updates to security.

-- 
You received this bug notification because you are a member of SE SRU
("STS") Sponsors, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/1989446

Title:
  [SRU] unable to boot guest with large memory when SEV is enabled on
  host

Status in grub2-unsigned package in Ubuntu:
  Fix Released
Status in grub2-unsigned source package in Focal:
  Triaged
Status in grub2-unsigned source package in Jammy:
  Triaged
Status in grub2-unsigned source package in Kinetic:
  Fix Committed

Bug description:
  [ Impact ]

  When booting a large memory guest (both focal and jammy) with 5.15 kernel on 
a SEV enabled host
  it fails to boot and shows the following error in dmesg:
  software IO TLB: Cannot allocate buffer

  But booting a Fedora36 guest works fine on a SEV enabled host

  With this kernel commit:
  
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=e998879d4fb7991856916972168cf27c0d86ed12

  SWIOTLB could allocate from 64MB to 1G top contiguous memory according
  to how much memory the system has

  in sev_setup_arch:

  size = total_mem * 6 / 100;
  size = clamp_val(size, IO_TLB_DEFAULT_SIZE, SZ_1G);
  swiotlb_adjust_size(size);

  Look into the memory block layout from Fedora grub, the available memory 
blocks are:
  [ 0.005879] memory[0x0] [0x1000-0x0009], 
0x0009f000 bytes flags: 0x0
  [ 0.005881] memory[0x1] [0x0010-0x7e9ecfff], 
0x7e8ed000 bytes flags: 0x0
  [ 0.005883] memory[0x2] [0x7eb1b000-0x7fb9afff], 
0x0108 bytes flags: 0x0
  [ 0.005885] memory[0x3] [0x7fbff000-0x7ffd], 
0x003e1000 bytes flags: 0x0
  [ 0.005886] memory[0x4] [0x0001-0x0004], 
0x0004 bytes flags: 0x0

  The biggest one is:
  [ 0.005881] memory[0x1] [0x0010-0x7e9ecfff], 
0x7e8ed000 bytes flags: 0x0

  The size is close to 2G and sufficient for SWIOTLB to allocate 1G
  contiguous memory

  Then we need to exclude reserved memory blocks overlapped with this region, 
below is the list
  [ 0.005892] reserved[0x2] [0x574a7000-0x59313fff], 
0x01e6d000 bytes flags: 0x0
  [ 0.005894] reserved[0x3] [0x7e133018-0x7e17e057], 
0x0004b040 bytes flags: 0x0
  [ 0.005896] reserved[0x4] [0x7e845018-0x7e845857], 
0x0840 bytes flags: 0x0
  [ 0.005897] reserved[0x5] [0x7ee95698-0x7ee95af7], 
0x0460 bytes flags: 0x0

  Now the biggest available range is
  [0x0010-0x574a7000]

  Before SWIOTLB allocates memory block, EFI also reserves some memory
  the one that overlapped with the above range is
  [ 0.005942] memblock_reserve: [0x7bfbe000-0x7bfddfff] 
efi_reserve_boot_services+0x8a/0xdb

  It’s fine that SWIOTLB can still allocate 1G contiguous memory from 
[0x0010-0x574a7000]:
  [ 1.089832] software IO TLB: mapped [mem 
0x174a7000-0x574a7000] (1024MB)

  But if we look into the memory block layout from Ubuntu grub, the available 
memory blocks are:
  [ 0.005833] memory[0x0] [0x1000-0x0009], 
0x0009f000 bytes flags: 0x0
  [ 0.005835] memory[0x1] [0x0010-0x7e9ecfff], 
0x7e8ed000 bytes flags: 0x0
  [ 0.005837] memory[0x2] [0x7eb1b000-0x7fb9afff], 
0x0108 bytes flags: 0x0
  [ 0.005838] memory[0x3] [0x7fbff000-0x7ffd], 
0x003e1000 bytes flags: 0x0
  [ 0.005840] memory[0x4] [0x0001-0x0004], 
0x0004 bytes flags: 0x0

  The biggest one is also:
  [ 0.005835] memory[0x1] [0x0010-0x7e9ecfff], 
0x7e8ed000 bytes flags: 0x0

  Then excluding the reserved memory blocks:
  [ 0.005846] reserved[0x2] [0x3a9ba000-0x3c7cdfff], 
0x01e14000 bytes flags: 0x0
  [ 0.005848] reserved[0x3] [0x7e133018-0x7e17e057], 
0x0004b040 bytes flags: 0x0
  [ 0.005849] reserved[0x4] [0x7e847018-0x7e847887], 
0x0870 bytes flags: 0x0
  [ 0.005851] reserved[0x5] [0x7ee95698-0x7ee95af7], 
0x0460 bytes flags: 0x0

  Now the biggest one is:
  [0x3e133018-0x7e133018]

  Then excluding EFI reserved memory block that overlapped with the above range:
  [ 0.005896] memblock_reserve: [0x7bfbe000-0x7bfddfff] 
efi_reserve_boot_services+0x8a/0xdb

  So now, the biggest contiguous memory becomes
  [0x3c7ce000-0x7bfbe000]

  Which is less than 1G, this is why SWIOTLB can not allocate 1G
  contiguous memory

  Thi

Re: [Sts-sponsors] [UNITTESTS] -b vault-refactor-cache lp:~igor-brovtsin/maas/+git/maas into -b master lp:~maas-committers/maas - TESTS PASS

2022-11-29 Thread MAAS Lander
Review: Approve

UNIT TESTS
-b vault-refactor-cache lp:~igor-brovtsin/maas/+git/maas into -b master 
lp:~maas-committers/maas

STATUS: SUCCESS
COMMIT: 554ae68aaed6560f04675559fc8ee88bcef766f0
-- 
https://code.launchpad.net/~igor-brovtsin/maas/+git/maas/+merge/433663
Your team MAAS Maintainers is requested to review the proposed merge of 
~igor-brovtsin/maas:vault-refactor-cache into maas:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp


Re: [Sts-sponsors] [Merge] ~igor-brovtsin/maas:vault-refactor-cache into maas:master

2022-11-29 Thread Igor Brovtsin
jenkins: !test
-- 
https://code.launchpad.net/~igor-brovtsin/maas/+git/maas/+merge/433663
Your team MAAS Maintainers is requested to review the proposed merge of 
~igor-brovtsin/maas:vault-refactor-cache into maas:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to : sts-sponsors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp