Avoid stopping services on upgrade

Fixed in https://github.com/canonical/landscape-client/pull/111

Thanks for outlining the motivation behind this, I think this is the right start.
But the discussion on LP: #2006402 has shown that people are rather unsure about it.
I realized my text got long so I split it in sections for readability, hope that helps.

#1 current situation

## Jammy

override_dh_installsystemd:
        dh_installsystemd --no-enable --no-start

Leading to
root@j:~# grep -C 5 landscape-client.service /var/lib/dpkg/info/landscape-client.p*
/var/lib/dpkg/info/landscape-client.postinst- # Migrate old sysvinit config to systemd
/var/lib/dpkg/info/landscape-client.postinst- DEFAULTS=/etc/default/landscape-client
/var/lib/dpkg/info/landscape-client.postinst- if [ -f $DEFAULTS ]; then
/var/lib/dpkg/info/landscape-client.postinst- RUN=$(. $DEFAULTS; echo $RUN)
/var/lib/dpkg/info/landscape-client.postinst- if [ "$RUN" = "1" ]; then
/var/lib/dpkg/info/landscape-client.postinst: systemctl enable landscape-client.service
/var/lib/dpkg/info/landscape-client.postinst- fi
/var/lib/dpkg/info/landscape-client.postinst- rm -f $DEFAULTS
/var/lib/dpkg/info/landscape-client.postinst- fi
/var/lib/dpkg/info/landscape-client.postinst- ;;
/var/lib/dpkg/info/landscape-client.postinst-
--
/var/lib/dpkg/info/landscape-client.postinst-fi
/var/lib/dpkg/info/landscape-client.postinst-
/var/lib/dpkg/info/landscape-client.postinst-# End automatically added section
/var/lib/dpkg/info/landscape-client.postinst-# Automatically added by dh_installsystemd/13.6ubuntu1
/var/lib/dpkg/info/landscape-client.postinst-if [ "$1" = "configure" ] || [ "$1" = "abort-upgrade" ] || [ "$1" = "abort-deconfigure" ] || [ "$1" = "abort-remove" ] ; then
/var/lib/dpkg/info/landscape-client.postinst: if deb-systemd-helper debian-installed 'landscape-client.service'; then
/var/lib/dpkg/info/landscape-client.postinst- # This will only remove masks created by d-s-h on package removal.
/var/lib/dpkg/info/landscape-client.postinst: deb-systemd-helper unmask 'landscape-client.service' >/dev/null || true
/var/lib/dpkg/info/landscape-client.postinst-
/var/lib/dpkg/info/landscape-client.postinst: if deb-systemd-helper --quiet was-enabled 'landscape-client.service'; then
/var/lib/dpkg/info/landscape-client.postinst- # Create new symlinks, if any.
/var/lib/dpkg/info/landscape-client.postinst: deb-systemd-helper enable 'landscape-client.service' >/dev/null || true
/var/lib/dpkg/info/landscape-client.postinst- fi
/var/lib/dpkg/info/landscape-client.postinst- fi
/var/lib/dpkg/info/landscape-client.postinst-
/var/lib/dpkg/info/landscape-client.postinst- # Update the statefile to add new symlinks (if any), which need to be cleaned
/var/lib/dpkg/info/landscape-client.postinst- # up on purge. Also remove old symlinks.
/var/lib/dpkg/info/landscape-client.postinst: deb-systemd-helper update-state 'landscape-client.service' >/dev/null || true
/var/lib/dpkg/info/landscape-client.postinst-fi
/var/lib/dpkg/info/landscape-client.postinst-# End automatically added section
/var/lib/dpkg/info/landscape-client.postinst-
/var/lib/dpkg/info/landscape-client.postinst-
/var/lib/dpkg/info/landscape-client.postinst-exit 0
--
/var/lib/dpkg/info/landscape-client.postrm-fi
/var/lib/dpkg/info/landscape-client.postrm-# End automatically added section
/var/lib/dpkg/info/landscape-client.postrm-# Automatically added by dh_installsystemd/13.6ubuntu1
/var/lib/dpkg/info/landscape-c...

#2 What do we want

Let me try to start this here in more detail.
I think we need a table here to list all desired behavior in the different possible states.
And then look for a solution providing all of that.
I know this is more intense now, but hopefully allows us to fix it just once :-)
(I hope this is monospace for all of you...)

This is what I think you described mitch, please correct me.

Action | Situation | Solution |
Pkg-install | no config, should not start | --no-enable --no-start |
------------|-----------------------------------|-----------------------------------|
Pkg-upgrade | if it runs, do not stop it | --no-enable --no-start |
            | because it would stay off | + --no-stop-on-upgrade |
------------|-----------------------------------|-----------------------------------|
Pkg-upgrade | if does not run, do not start | --no-enable --no-start |
------------|-----------------------------------|-----------------------------------|

#3 What do we really want?

Now the problem with that is that just using the predefined `--no-enable --no-start` or the same with `--no-stop-on-upgrade` won't do all you need.

The table should IMHO actually look like (please really correct me if that is wrong):

Action | Situation | Solution |
Pkg-install | no config, should not start | --no-enable --no-start |
------------|-----------------------------------|-----------------------------------|
Pkg-install | config present, e.g. by config | neither of --no-enable --no-start |
            | or from old install or any | would be allowed |
            | orchestration SW, should start | |
------------|-----------------------------------|-----------------------------------|
Pkg-upgrade | if it runs, restart it to pick up | --no-stop-on-upgrade not allowed |
            | the new code, imagine you have a | |
            | CVE - you need to restart | |
------------|-----------------------------------|-----------------------------------|
Pkg-upgrade | if not configure, do not touch it | need to check if configured |
------------|-----------------------------------|-----------------------------------|

There might be more, feel free to add them to help any sponsor to think with you about it.

# Suggestion to think about

So IMHO you have a service that is neither never nor always (re-)started.
And IMHO the arguments to dh_installsystemd are never enough to express that.

But systemd has mechanisms for such, for example [1].

I haven't dealt with all your config.
There is /var/lib/landscape/client/ and /etc/landscape/client.conf and sometimes landscape-client.conf.
That is up to you to know better which might be good.

Sadly most of the checks are for existence and your files all exist on install, they are just not fully ready yet. I registered a client and compared the config pre/post attach.

I realize that having or not-having account_name in /etc/landscape/client.conf might be the perfect condition. But systemd can't check on file content AFAIK. What would you think of using a /etc/landscape/registered or anything in /var/lib/landscape/client that you think you can rely on to be present "after config" but "before service start".

Gladly systemd adds a special way for everything :-)
For "advanced" conditions there is [2] which you might use here.
And you have - in the new version - gladly `landscape-config --is-registered`

I'm not sure how this internally works, --is-registered needs to be able to return true after being configured, but before the service has been started. I think you check /var/lib/landscape/client/broker.bpickle which might only exist after the first start, but I'm sure you could work something out that solves this and then would really cover all use-cases better.

As an example something like the following might cover all of the above cases much better (please check and amend with your subject matter expertise):

$ git diff
diff --git a/debian/landscape-client.service b/debian/landscape-client.service
index 6dc721a3..aa5c0261 100644
--- a/debian/landscape-client.service
+++ b/debian/landscape-client.service
@@ -11,6 +11,7 @@ WantedBy=multi-user.target
 [Service]
 Type=simple
 Group=landscape
+ExecCondition=/usr/bin/landscape-config --is-registered
 ExecStart=/usr/bin/landscape-client
 # Don't kill cgroup as child dpkg may restart the service during an upgrade.
 KillMode=process
diff --git a/debian/rules b/debian/rules
index ad30f5c9..d6a156c6 100755
--- a/debian/rules
+++ b/debian/rules
@@ -23,4 +23,4 @@ override_dh_auto_install:
        install -D -o root -g root -m 755 apt-update/apt-update $(CURDIR)/$(root_dir)$(LIBDIR)/apt-update

 override_dh_installsystemd:
- dh_installsystemd --no-enable --no-start --no-stop-on-upgrade
+ dh_installsystemd

[1]: https://www.freedesktop.org/software/systemd/man/systemd.unit.html#ConditionFileNotEmpty=
[2]: https://www.freedesktop.org/software/systemd/man/systemd.service.html#ExecCondition=

Thanks, Christian. I really like the idea of using ExecCondition and going back to default dh_installsystemd.

A little bit of info regarding landscape-config --is-registered and configuration/registration in general. When landscape-config is run to register landscape, after /etc/landscape/client.conf is updated with the provided values, landscape-config runs `systemctl restart landscape-client.service` is run to complete the registration process with the server. This is the "first start" of the service, and if it doesn't happen, then there's no possibility of registration being completed successfully.

Basically /var/lib/landscape/client/broker.bpickle existing is a prerequisite of being registered. There's no way for --is-registered to be able to return true before the service has been started because the service starting is required for registration in the first place.

I'm going to go ahead and test out the proposed patch.

Testing summary, tests run on jammy:

test 1: installed version 19.12-0ubuntu13, registered, service is running, upgraded to version 23
result: version 23 is now the running service

test 2: installed version 19.12-0ubuntu13, registered, systemctl stop landscape-client, upgraded to version 23
result: version 23 is now the running service

test 3: installed version 19.12-0ubuntu13, registered, systemctl stop landscape-client, systemctl disable landscape-client, upgraded to version 23
result: service is not running

test 4: installed version 19.23-0ubuntu13, did not register, service is not running, upgraded to version 23
result: service is not running

Repeated tests on focal with same results (expect upgrading from version 19.12-0ubuntu4.3).

The version I used for these tests can be found in this PPA: https://launchpad.net/~mitchburton/+archive/ubuntu/landscape-client-sru-tests

I think this is almost an adequate solution, but it's still concerning that a manually stopped (but not disabled) service is restarted on upgrade.

I'm happy that it seems to solve all your cases

I do not think that remaining concern of yours is a real concern we need to have.
That is the standard behavior and usually "unless you have a reason, you want to be normal".
Here an example of a service people consider mundane:

root@m:~# apt install apache2
root@m:~# systemctl status apache2 | head -n 3
● apache2.service - The Apache HTTP Server
     Loaded: loaded (/lib/systemd/system/apache2.service; enabled; preset: enabled)
     Active: active (running) since Mon 2023-07-24 06:04:24 UTC; 1min 29s ago
root@m:~# systemctl stop apache2
root@m:~# systemctl status apache2 | head -n 3
○ apache2.service - The Apache HTTP Server
     Loaded: loaded (/lib/systemd/system/apache2.service; enabled; preset: enabled)
     Active: inactive (dead) since Mon 2023-07-24 06:06:15 UTC; 2s ago
root@m:~# apt install --reinstall apache2
...
root@m:~# systemctl status apache2 | head -n 3
● apache2.service - The Apache HTTP Server
     Loaded: loaded (/lib/systemd/system/apache2.service; enabled; preset: enabled)
     Active: active (running) since Mon 2023-07-24 06:06:34 UTC; 5s ago

So it behaves just like yours.
Remember, stopped services would also come back up on reboot.
People know (or need to learn and understand) that "stop" really just means "stop now".

And your own tests confirm that "disable" does well what it is intended to do.

It is up to you, but my gut feeling says that with this current proposed solution it would be good?
If you agree you want to get that uploaded to mantic and then augment the ongoing SRUs to include the same.

Right, thanks for making the stop/disabled difference incredibly clear. I'm happy with the proposed solution and will get it merged upstream and update the SRUs to include it. Thanks again!

Since this is going to be fixed via SRU I created tasks for Focal and Jammy. Could you please add the SRU template to the bug description?

https://wiki.ubuntu.com/StableReleaseUpdates#SRU_Bug_Template

this is looking good, but the test plan in the description needs to include a description of the expected behavior for each test, not just stating what you will run.

I've updated the description with the expected behaviour for each test.

Hello Mitch, or anyone else affected,

Accepted landscape-client into jammy-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/landscape-client/23.02-0ubuntu1~22.04.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-jammy to verification-done-jammy. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-jammy. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Hello Mitch, or anyone else affected,

Accepted landscape-client into focal-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/landscape-client/23.02-0ubuntu1~20.04.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-focal to verification-done-focal. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-focal. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

In the process of verification testing, I discovered a gap in my previous testing: I never tested simply installing the new version of the package and running `landscape-config`.

Unfortunately `landscape-config` relies on starting the service to register and cannot because of the new ExecCondition, which requires registration to already have happened! I can think of a few different ways to resolve this:

  1. restructure `landscape-config` so that it doesn't need to start the service to register (nontrivial amount of work)
  2. modify `landscape-config --is-registered` so that it is aware that we are currently in the `landscape-config` wizard
  3. change the ExecCondition to something like
     /usr/bin/landscape-config --is-registered || ! systemctl is-active landscape-client.service
     so that we can also start the service when it is manually enabled, such as the case where we are not registered but `landscape-config` is being run

This fix is not present in lunar and later (mantic, noble), reopening and adding tasks.

I've created MRs for lunar, mantic, and noble. These MRs also address LP: #2040189

This bug was fixed in the package landscape-client - 23.08-0ubuntu4

---------------
landscape-client (23.08-0ubuntu4) noble; urgency=medium

  * d/p/0003-fix-cpuinfo-and-tests.patch: fix ARM and RISCV cpuinfo parsing;
    fix environment var-modifying tests, increase timeout on tests that can
    be slow on these arches (LP: #2046620).

 -- Mitch Burton <email address hidden> Thu, 04 Jan 2023 17:06:26 -0800

Hello Mitch, or anyone else affected,

Accepted landscape-client into jammy-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/landscape-client/23.02-0ubuntu1~22.04.2 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-jammy to verification-done-jammy. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-jammy. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

the mantic fix is still needed?

Hello Mitch, or anyone else affected,

Accepted landscape-client into focal-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/landscape-client/23.02-0ubuntu1~20.04.2 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-focal to verification-done-focal. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-focal. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

>the mantic fix is still needed?

Yes. I will have a merge request out for mantic shortly.

Verification on focal with version 23.02-0ubuntu1~20.04.2

test 1:
 - installed version 19.12-0ubuntu4.3
 - registered with Landscape Server
 - service is running:

ubuntu@alive-squid:~$ systemctl is-active landscape-client
active

ubuntu@alive-squid:~$ pgrep landscape
4315
4317
4318
4319

 - upgraded to version 23.02-0ubuntu1~20.04.2

expected behaviour: landscape-client service restarts, running the new version (23.02-0ubuntu1~20.04.2)
observed behaviour: landscape-client service restarted, and is running the new version.

ubuntu@alive-squid:~$ pgrep landscape
5585
5714
5715
5717

test 2:
 - installed version 19.12-0ubuntu4.3
 - registered with Landscape Server
 - ran `systemctl stop landscape-client`

ubuntu@positive-raven:~$ sudo systemctl stop landscape-client
ubuntu@positive-raven:~$ pgrep landscape
(no output)

 - upgraded to version 23.02-0ubuntu1~20.04.2

expected behaviour: landscape-client service restarts, running the new version (23.02-0ubuntu1~20.04.2)
observed behaviour: landscape-client service restarted, and is running the new version.

ubuntu@positive-raven:~$ pgrep landscape
5165
5286
5287
5289

test 3:
 - installed version 19.12-0ubuntu4.3
 - registered with Landscape Server
 - ran `systemctl stop landscape-client`
 - ran `systemctl disable landscape-client`

ubuntu@thorough-honeybee:~$ sudo systemctl disable landscape-client
Removed /etc/systemd/system/multi-user.target.wants/landscape-client.service.

 - upgraded to version 23.02-0ubuntu1~20.04.2

expected behaviour: landscape-client service does not start (as it is disabled), but when enabled, the new version (23.02-0ubuntu1~20.04.2) is running.
observed behaviour: landscape-client service did not start

ubuntu@thorough-honeybee:~$ systemctl is-active landscape-client
inactive
ubuntu@thorough-honeybee:~$ systemctl is-enabled landscape-client
disabled

After systemctl enable and systemctl start landscape-client, it is running the new version.

ubuntu@thorough-honeybee:~$ sudo systemctl enable landscape-client
Created symlink /etc/systemd/system/multi-user.target.wants/landscape-client.service → /lib/systemd/system/landscape-client.service.
ubuntu@thorough-honeybee:~$ sudo systemctl start landscape-client
ubuntu@thorough-honeybee:~$ pgrep landscape
5375
5377
5378
5379

test 4:
 - installed version 19.12-0ubuntu4.3
 - did not register with Landscape Server
 - service is not running

ubuntu@suited-stallion:~$ sudo systemctl is-active landscape-client
inactive
ubuntu@suited-stallion:~$ sudo systemctl is-enabled landscape-client
disabled

 - upgraded to version 23.02-0ubuntu1~20.04.2

expected behaviour: landscape-client service never runs, registration has not occurred.
observed behaviour: landscape-client service did not run, registration has not occurred.

ubuntu@suited-stallion:~$ sudo systemctl is-active landscape-client
inactive
ubuntu@suited-stallion:~$ sudo systemctl is-enabled landscape-client
disabled

Verification on jammy with version 23.02-0ubuntu1~22.04.2

test 1:
 - installed version 19.12-0ubuntu13
 - registered with Landscape Server
 - service is running:

ubuntu@allowed-beetle:~$ systemctl is-active landscape-client
active

ubuntu@allowed-beetle:~$ pgrep landscape
2276
2277
2278
2279

 - upgraded to version 23.02-0ubuntu1~22.04.2

ubuntu@allowed-beetle:~$ sudo apt install landscape-client
...
Restarting services...
 systemctl restart landscape-client.service
...

expected behaviour: landscape-client service restarts, running the new version (23.02-0ubuntu1~22.04.2)
observed behaviour: landscape-client service restarted, and is running the new version.

ubuntu@allowed-beetle:~$ pgrep landscape
3529
3550
3551
3552

test 2:
 - installed version 19.12-0ubuntu13
 - registered with Landscape Server
 - ran `systemctl stop landscape-client`

ubuntu@flowing-glowworm:~$ sudo systemctl stop landscape-client
ubuntu@flowing-glowworm:~$ pgrep landscape
(no output)

 - upgraded to version 23.02-0ubuntu1~22.04.2

expected behaviour: landscape-client service restarts, running the new version (23.02-0ubuntu1~22.04.2)
observed behaviour: landscape-client service restarted, and is running the new version.

ubuntu@flowing-glowworm:~$ pgrep landscape
3090
3113
3114
3115

test 3:
 - installed version 19.12-0ubuntu13
 - registered with Landscape Server
 - ran `systemctl stop landscape-client`
 - ran `systemctl disable landscape-client`

ubuntu@caring-manatee:~$ sudo systemctl disable landscape-client
Removed /etc/systemd/system/multi-user.target.wants/landscape-client.service.

 - upgraded to version 23.02-0ubuntu1~22.04.2

expected behaviour: landscape-client service does not start (as it is disabled), but when enabled, the new version (23.02-0ubuntu1~22.04.2) is running.
observed behaviour: landscape-client service did not start

ubuntu@caring-manatee:~$ systemctl is-active landscape-client
inactive
ubuntu@caring-manatee:~$ systemctl is-enabled landscape-client
disabled

After systemctl enable and systemctl start landscape-client, it is running the new version.

ubuntu@caring-manatee:~$ sudo systemctl enable landscape-client
Created symlink /etc/systemd/system/multi-user.target.wants/landscape-client.service → /lib/systemd/system/landscape-client.service.
ubuntu@caring-manatee:~$ sudo systemctl start landscape-client
ubuntu@caring-manatee:~$ pgrep landscape
3287
3288
3289
3290

test 4:
 - installed version 19.12-0ubuntu13
 - did not register with Landscape Server
 - service is not running

ubuntu@sacred-buzzard:~$ systemctl is-active landscape-client
inactive
ubuntu@sacred-buzzard:~$ systemctl is-enabled landscape-client
disabled

 - upgraded to version 23.02-0ubuntu1~22.04.2

expected behaviour: landscape-client service never runs, registration has not occurred.
observed behaviour: landscape-client service did not run, registration has not occurred.

ubuntu@sacred-buzzard:~$ systemctl is-active landscape-client
inactive
ubuntu@sacred-buzzard:~$ systemctl is-enabled landscape-client
disabled

Hello Mitch, or anyone else affected,

Accepted landscape-client into mantic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/landscape-client/23.08-0ubuntu1.2 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-mantic to verification-done-mantic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-mantic. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Verification on mantic with version 23.08-0ubuntu1.2

test 1:
 - installed version 23.08-0ubuntu1
 - registered with Landscape Server
 - service is running:

ubuntu@useful-tuna:~$ systemctl is-active landscape-client
active

ubuntu@useful-tuna:~$ pgrep landscape
3945
3946
3947
3948

 - upgraded to version 23.08-0ubuntu1.2

expected behaviour: landscape-client service restarts, running the new version (23.08-0ubuntu1.2)
observed behaviour: landscape-client service restarted, and is running the new version.

ubuntu@useful-tuna:~$ pgrep landscape
5110
5128
5130
5131

test 2:
 - installed version 23.08-0ubuntu1
 - registered with Landscape Server
 - ran `systemctl stop landscape-client`

ubuntu@gentle-sloth:~$ sudo systemctl stop landscape-client
ubuntu@gentle-sloth:~$ pgrep landscape
(no output)

 - upgraded to version 23.08-0ubuntu1.2

expected behaviour: landscape-client service restarts, running the new version (23.08-0ubuntu1.2)
observed behaviour: landscape-client service restarted, and is running the new version.

ubuntu@gentle-sloth:~$ pgrep landscape
4638
4657
4658
4659

test 3:
 - installed version 23.08-0ubuntu1
 - registered with Landscape Server
 - ran `systemctl stop landscape-client`
 - ran `systemctl disable landscape-client`

ubuntu@immense-peacock:~$ sudo systemctl disable landscape-client
Removed "/etc/systemd/system/multi-user.target.wants/landscape-client.service".

 - upgraded to version 23.08-0ubuntu1.2

expected behaviour: landscape-client service does not start (as it is disabled), but when enabled, the new version (23.08-0ubuntu1.2) is running.
observed behaviour: landscape-client service did not start

ubuntu@immense-peacock:~$ sudo systemctl is-active landscape-client
inactive
ubuntu@immense-peacock:~$ sudo systemctl is-enabled landscape-client
disabled

After systemctl enable and systemctl start landscape-client, it is running the new version.

ubuntu@immense-peacock:~$ sudo systemctl enable landscape-client
Created symlink /etc/systemd/system/multi-user.target.wants/landscape-client.service → /lib/systemd/system/landscape-client.service.
ubuntu@immense-peacock:~$ sudo systemctl start landscape-client
ubuntu@immense-peacock:~$ pgrep landscape
4807
4808
4809
4810

test 4:
 - installed version 23.08-0ubuntu1
 - did not register with Landscape Server
 - service is not running

ubuntu@exciting-bream:~$ sudo systemctl is-active landscape-client
inactive
ubuntu@exciting-bream:~$ sudo systemctl is-enabled landscape-client
disabled

 - upgraded to version 23.08-0ubuntu1.2

expected behaviour: landscape-client service never runs, registration has not occurred.
observed behaviour: landscape-client service did not run, registration has not occurred.

ubuntu@exciting-bream:~$ sudo systemctl is-active landscape-client
inactive
ubuntu@exciting-bream:~$ sudo systemctl is-enabled landscape-client
disabled