fw update: tests for software update

Tests for the software update flow.

Minimal PLDM packages are constructed in testcases and the update flow
is run on software instance.

These tests should chec

fw update: tests for software update

Tests for the software update flow.

Minimal PLDM packages are constructed in testcases and the update flow
is run on software instance.

These tests should check that the common code behaves as outlined in the
design [1]

References:
[1] https://github.com/openbmc/docs/blob/master/designs/code-update.md

Change-Id: Ibf473999b692a85dd86e4ab57a9a4c322257fc40
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>

show more ...

fw update: tests for device

Tests for the device update flow.

Minimal PLDM packages are constructed in testcases and the update flow
is run on device instance.

These tests should check that the co

fw update: tests for device

Tests for the device update flow.

Minimal PLDM packages are constructed in testcases and the update flow
is run on device instance.

These tests should check that the common code behaves as outlined in the
design [1]

References:
[1] https://github.com/openbmc/docs/blob/master/designs/code-update.md

Change-Id: I8f12839afd47ef3403a80439af54fedcc00f10be
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>

show more ...

test: common: software associations: remove NOLINT

remove `clang-analyzer-core.uninitialized.Branch` in a testcase.

Apparently the NOLINT is not required there, not sure if it was due to
clang upda

test: common: software associations: remove NOLINT

remove `clang-analyzer-core.uninitialized.Branch` in a testcase.

Apparently the NOLINT is not required there, not sure if it was due to
clang update or if it's been resolved in later revisions of the linked
patch. Anyways we can drop it now.

Tested: clang-tidy passes

Fixes: ade5c5aad34fa313b1bdd88577f44ba6f64ef43c

Change-Id: Ibba3a10db586b3096880fecba68ebc797b6ac077
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>

show more ...

test: common: software

Test API of `class Software`, focus on initial state on construction.

Change-Id: I20350748d02f62cfb8371fee68d43ebbdb91609c
Signed-off-by: Alexander Hansen <alexander.hansen@9

test: common: software

Test API of `class Software`, focus on initial state on construction.

Change-Id: I20350748d02f62cfb8371fee68d43ebbdb91609c
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>

show more ...

test: common: software version

Test Software Version DBus API.

Change-Id: I287b7e7957199463ec80bda8f71bf881bc0e5cb8
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>

test: common: associations

Test software associations in common code.
Write necessary wrappers in example device to use the protected members.

Change-Id: If7c38f12472699672ed8a4c1b3e1c99c398cdba5
S

test: common: associations

Test software associations in common code.
Write necessary wrappers in example device to use the protected members.

Change-Id: If7c38f12472699672ed8a4c1b3e1c99c398cdba5
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>

show more ...

All: add purpose parameter to setVersion calls

All calls to setVersion now include a purpose argument, changing the
purpose string from "unknown" to "other".

Change-Id: I201db1719333b51deead2dd99c9

All: add purpose parameter to setVersion calls

All calls to setVersion now include a purpose argument, changing the
purpose string from "unknown" to "other".

Change-Id: I201db1719333b51deead2dd99c9f637c782efd6c
Signed-off-by: Daniel Hsu <Daniel-Hsu@quantatw.com>

show more ...

test: common: software config

Tests for the common configuration all updaters receive from EM.

Change-Id: Ic242b5fbaa88a1b10ec9b9e97e084e1e85e7e9e0
Signed-off-by: Alexander Hansen <alexander.hansen

test: common: software config

Tests for the common configuration all updaters receive from EM.

Change-Id: Ic242b5fbaa88a1b10ec9b9e97e084e1e85e7e9e0
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>

show more ...

test: common: random software id

Test that the random software id matches our regex.

Change-Id: I0eb9b5c58b993ef6172b059daec086872e7656fb
Signed-off-by: Alexander Hansen <alexander.hansen@9elements

test: common: random software id

Test that the random software id matches our regex.

Change-Id: I0eb9b5c58b993ef6172b059daec086872e7656fb
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>

show more ...

clang-tidy: remove some NOLINTS

Since readability-static-accessed-through-instance check has been
disabled in [1], the NOLINTs can be removed.

References:

[1] https://gerrit.openbmc.org/c/openbmc/

clang-tidy: remove some NOLINTS

Since readability-static-accessed-through-instance check has been
disabled in [1], the NOLINTs can be removed.

References:

[1] https://gerrit.openbmc.org/c/openbmc/phosphor-bmc-code-mgmt/+/80392

Change-Id: If7e80a0bc447bfbd054b6c007d230c2d14a54f41
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>
Signed-off-by: Jagpal Singh Gill <paligill@gmail.com>

show more ...

meson: apply meson format

since [1] was merged in openbmc-build-scripts, meson format is required
to pass CI.

References:
[1] https://gerrit.openbmc.org/c/openbmc/openbmc-build-scripts/+/77742

Cha

meson: apply meson format

since [1] was merged in openbmc-build-scripts, meson format is required
to pass CI.

References:
[1] https://gerrit.openbmc.org/c/openbmc/openbmc-build-scripts/+/77742

Change-Id: I66206066c41dd3d6225855d0530c72f5089a964b
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>

show more ...

fixup: test: example device

- Fix the update progress loop in
test/common/exampledevice/example_device.cpp.
The update progress should be incremented there.

- Move a comment to the relevant lin

fixup: test: example device

- Fix the update progress loop in
test/common/exampledevice/example_device.cpp.
The update progress should be incremented there.

- Move a comment to the relevant lines in
test/common/exampledevice/example_updater_main.cpp

Tested: Unit tests pass.

Change-Id: I5fa908e1c6f95eaaa2710853c065fc341a30035f
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>

show more ...

fw update: implement example device

Introduce example-device and example-code-updater, which are used as
examples for how to implement devices and code updaters according to the
design [1]

Tested:

fw update: implement example device

Introduce example-device and example-code-updater, which are used as
examples for how to implement devices and code updaters according to the
design [1]

Tested:

The example code updater allows us to perform manual testing without
a dependency on any specific device.

Running the example code updater results in follow dbus output:

```
busctl --full --no-pager tree xyz.openbmc_project.ExampleCodeUpdater

└─ /xyz
└─ /xyz/openbmc_project
└─ /xyz/openbmc_project/software
└─ /xyz/openbmc_project/software/ExampleSoftware_4081

busctl --full --no-pager introspect xyz.openbmc_project.ExampleCodeUpdater /xyz/openbmc_project/software/ExampleSoftware_4081

CodeUpdater /xyz/openbmc_project/software/ExampleSoftware_4081
NAME TYPE SIGNATURE RESULT/VALUE FLAGS
...
xyz.openbmc_project.Software.Activation interface - - -
.Activation property s "xyz.openbmc_project.Software.Activation.Activations.Active" emits-change writable
.RequestedActivation property s "xyz.openbmc_project.Software.Activation.RequestedActivations.None" emits-change writable
xyz.openbmc_project.Software.Update interface - - -
.StartUpdate method hs o -
.AllowedApplyTimes property as 1 "xyz.openbmc_project.Software.ApplyTime.RequestedApplyTimes.OnReset" emits-change
xyz.openbmc_project.Software.Version interface - - -
.Purpose property s "xyz.openbmc_project.Software.Version.VersionPurpose.Unknown" emits-change writable
.Version property s "v1.0"
```

References:
[1] https://github.com/openbmc/docs/blob/master/designs/code-update.md

Change-Id: I2bad241b3102e58eda5139174791adda82f6ca95
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>

show more ...