Merge ~lamoura/ubuntu/+source/ubuntu-advantage-tools:upload-29.3-mantic into ubuntu/+source/ubuntu-advantage-tools:ubuntu/devel

Proposed by Lucas Albuquerque Medeiros de Moura
Status: Merged
Merged at revision: 8de7686c99e692d69a8ff4a74772b8be3570edd6
Proposed branch: ~lamoura/ubuntu/+source/ubuntu-advantage-tools:upload-29.3-mantic
Merge into: ubuntu/+source/ubuntu-advantage-tools:ubuntu/devel
Diff against target: 92 lines (+42/-2)
4 files modified
debian/changelog (+7/-0)
features/api.feature (+33/-0)
uaclient/api/u/pro/status/enabled_services/v1.py (+1/-1)
uaclient/version.py (+1/-1)
Reviewer Review Type Date Requested Status
Sergio Durigan Junior (community) Approve
Ubuntu Sponsors Pending
Canonical Server Reporter Pending
Review via email: mp+450057@code.launchpad.net

Description of the change

Release 29.3 of Ubuntu Pro client addressing a bug on importing an API endpoint

To post a comment you must log in.
Revision history for this message
Sergio Durigan Junior (sergiodj) wrote :

Thanks, Lucas.

I see that the builds have passed. The changes are simple, so +1 from me.

Uploaded:

$ dput ubuntu-advantage-tools_29.3_source.changes
Trying to upload package to ubuntu
Checking signature on .changes
gpg: /home/sergio/work/ubuntu-advantage-tools/ubuntu-advantage-tools_29.3_source.changes: Valid signature from 106DA1C8C3CBBF14
Checking signature on .dsc
gpg: /home/sergio/work/ubuntu-advantage-tools/ubuntu-advantage-tools_29.3.dsc: Valid signature from 106DA1C8C3CBBF14
Uploading to ubuntu (via ftp to upload.ubuntu.com):
  Uploading ubuntu-advantage-tools_29.3.dsc: done.
  Uploading ubuntu-advantage-tools_29.3.tar.xz: done.
  Uploading ubuntu-advantage-tools_29.3_source.buildinfo: done.
  Uploading ubuntu-advantage-tools_29.3_source.changes: done.
Successfully uploaded packages.

review: Approve
Revision history for this message
Sergio Durigan Junior (sergiodj) wrote :

Also uploaded to X/B/F/J/L.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/debian/changelog b/debian/changelog
index 1378858..308c1d7 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
1ubuntu-advantage-tools (29.3) mantic; urgency=medium
2
3 * api: fix circular import that prevents enabled_services
4 endpoint from being imported
5
6 -- Lucas Moura <lucas.moura@canonical.com> Fri, 25 Aug 2023 14:26:37 -0300
7
1ubuntu-advantage-tools (29.2) mantic; urgency=medium8ubuntu-advantage-tools (29.2) mantic; urgency=medium
29
3 * d/ubuntu-advantage-tools.postinst:10 * d/ubuntu-advantage-tools.postinst:
diff --git a/features/api.feature b/features/api.feature
index f7ce206..31a8691 100644
--- a/features/api.feature
+++ b/features/api.feature
@@ -1,6 +1,39 @@
1Feature: Client behaviour for the API endpoints1Feature: Client behaviour for the API endpoints
22
3 @series.all3 @series.all
4 @uses.config.machine_type.any
5 @uses.config.machine_type.lxd-container
6 Scenario Outline: all API endpoints can be imported individually
7 Given a `<release>` `<machine_type>` machine with ubuntu-advantage-tools installed
8 When I run `python3 -c "from uaclient.api.u.pro.attach.auto.configure_retry_service.v1 import configure_retry_service"` as non-root
9 When I run `python3 -c "from uaclient.api.u.pro.attach.auto.full_auto_attach.v1 import full_auto_attach"` as non-root
10 When I run `python3 -c "from uaclient.api.u.pro.attach.auto.should_auto_attach.v1 import should_auto_attach"` as non-root
11 When I run `python3 -c "from uaclient.api.u.pro.attach.magic.initiate.v1 import initiate"` as non-root
12 When I run `python3 -c "from uaclient.api.u.pro.attach.magic.revoke.v1 import revoke"` as non-root
13 When I run `python3 -c "from uaclient.api.u.pro.attach.magic.wait.v1 import wait"` as non-root
14 When I run `python3 -c "from uaclient.api.u.pro.packages.summary.v1 import summary"` as non-root
15 When I run `python3 -c "from uaclient.api.u.pro.packages.updates.v1 import updates"` as non-root
16 When I run `python3 -c "from uaclient.api.u.pro.security.fix.cve.plan.v1 import plan"` as non-root
17 When I run `python3 -c "from uaclient.api.u.pro.security.fix.usn.plan.v1 import plan"` as non-root
18 When I run `python3 -c "from uaclient.api.u.pro.security.status.livepatch_cves.v1 import livepatch_cves"` as non-root
19 When I run `python3 -c "from uaclient.api.u.pro.security.status.reboot_required.v1 import reboot_required"` as non-root
20 When I run `python3 -c "from uaclient.api.u.pro.status.enabled_services.v1 import enabled_services"` as non-root
21 When I run `python3 -c "from uaclient.api.u.pro.status.is_attached.v1 import is_attached"` as non-root
22 When I run `python3 -c "from uaclient.api.u.pro.version.v1 import version"` as non-root
23 When I run `python3 -c "from uaclient.api.u.security.package_manifest.v1 import package_manifest"` as non-root
24 When I run `python3 -c "from uaclient.api.u.unattended_upgrades.status.v1 import status"` as non-root
25 When I run `python3 -c "from uaclient.api.u.apt_news.current_news.v1 import current_news"` as non-root
26
27 Examples: ubuntu release
28 | release | machine_type |
29 | xenial | lxd-container |
30 | bionic | lxd-container |
31 | focal | lxd-container |
32 | jammy | lxd-container |
33 | lunar | lxd-container |
34 | mantic | lxd-container |
35
36 @series.all
4 @uses.config.machine_type.lxd-container37 @uses.config.machine_type.lxd-container
5 Scenario Outline: API invalid endpoint or args38 Scenario Outline: API invalid endpoint or args
6 Given a `<release>` machine with ubuntu-advantage-tools installed39 Given a `<release>` machine with ubuntu-advantage-tools installed
diff --git a/uaclient/api/u/pro/status/enabled_services/v1.py b/uaclient/api/u/pro/status/enabled_services/v1.py
index 3e2e398..440c1f2 100644
--- a/uaclient/api/u/pro/status/enabled_services/v1.py
+++ b/uaclient/api/u/pro/status/enabled_services/v1.py
@@ -11,7 +11,6 @@ from uaclient.data_types import (
11 StringDataValue,11 StringDataValue,
12 data_list,12 data_list,
13)13)
14from uaclient.entitlements.entitlement_status import UserFacingStatus
1514
1615
17class EnabledService(DataObject):16class EnabledService(DataObject):
@@ -48,6 +47,7 @@ def enabled_services() -> EnabledServicesResult:
4847
49def _enabled_services(cfg: UAConfig) -> EnabledServicesResult:48def _enabled_services(cfg: UAConfig) -> EnabledServicesResult:
50 from uaclient.entitlements import ENTITLEMENT_CLASSES49 from uaclient.entitlements import ENTITLEMENT_CLASSES
50 from uaclient.entitlements.entitlement_status import UserFacingStatus
5151
52 if not _is_attached(cfg).is_attached:52 if not _is_attached(cfg).is_attached:
53 return EnabledServicesResult(enabled_services=[])53 return EnabledServicesResult(enabled_services=[])
diff --git a/uaclient/version.py b/uaclient/version.py
index 5191f37..4d61635 100644
--- a/uaclient/version.py
+++ b/uaclient/version.py
@@ -15,7 +15,7 @@ from uaclient.defaults import CANDIDATE_CACHE_PATH, UAC_RUN_PATH
15from uaclient.exceptions import ProcessExecutionError15from uaclient.exceptions import ProcessExecutionError
16from uaclient.system import subp16from uaclient.system import subp
1717
18__VERSION__ = "29.2"18__VERSION__ = "29.3"
19PACKAGED_VERSION = "@@PACKAGED_VERSION@@"19PACKAGED_VERSION = "@@PACKAGED_VERSION@@"
2020
21CANDIDATE_REGEX = r"Candidate: (?P<candidate>.*?)\n"21CANDIDATE_REGEX = r"Candidate: (?P<candidate>.*?)\n"

Subscribers

People subscribed via source and target branches

to status/vote changes: