Signed-off-by: Esben Haabendal <esben@geanix.com>

With this in place, we are ready to start converting OpenHarmony system
abilities (started with sa_main) to systemd services using Type=notify.

Signed-off-by: Esben Haabendal <esben@geanix.com>

Convert hilogd into a systemd service

- Patch hilogd to use sd_notify(3) to send ready notification to systemd, and
  thus use as a Type=notify service.
- Add log and logd users and groups, with similar uid and gid as in OHOS.
- Place Unix sockets in /run/openharmony/hilog instead of /dev/unix/socket,
  enabling secure creaton and managing of these.
- Use systemd.socket units for the two Unix sockets, to allow creating with
  desired uid/gid and permissions, and allowing to use hilogd with systemd
  socket activation

Signed-off-by: Esben Haabendal <esben@geanix.com>

Signed-off-by: Esben Haabendal <esben@geanix.com>

Signed-off-by: Esben Haabendal <esben@geanix.com>

This stops placing temporary files and Unix sockets in /dev, and use /run
instead.  This aligns better with FHS and makes it easier to handle permissions
safely if/when we decide to change uid/gid of the param service.

Signed-off-by: Esben Haabendal <esben@geanix.com>

This ensures that when used in a systemd service, other units that depend on it
will only been started when the param_service is actually ready to respond to
requests.

Signed-off-by: Esben Haabendal <esben@geanix.com>

With ICU data file installed in /system/usr/ohos_icu this crash doesn't happen,
so we can safely drop this workaround.

Signed-off-by: Esben Haabendal <esben@geanix.com>

Signed-off-by: Esben Haabendal <esben@geanix.com>

add and apply actsfeatureabilitytest-wait-between-testcases.patch

test/xts/acts/ActsFeatureAbilityTest: wait 8 seconds between each testcase

sometimes feature abilities fail to start because the previous testcase
still has't completely started or terminated its feature ability.
Increasing sleep time from 1 seconds to 8 seconds between each testcase.

Upstream-Status: Pending

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

test/xts/acts/ActsGetWantTest: wait 10 seconds between each testcase

sometimes feature abilities fail to start because the previous testcase
still has't completely started or terminated its feature ability.
Adding a 10 seconds wait time between each testcase.

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

- increment testcase timeout and START_ABILITY_TIMEOUT for actsfeatureabilitytest
- adjust testcase timeout from 60 seconds to 20 seconds
- adjust actsbmsmoduleusagerecordtest START_ABILITY_TIMEOUT from 30 to 15 seconds
- rename patch files

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

test/xts/acts: increase testsuite timeouts

sometimes testsuites execution are timing out because
the execution takes longer to finish.
Increasing some testsuite timeouts.

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

foundation/ace/ace_engine: disable graphics

The graphics rendering is slowing the system making some ACTS test suites to timeout.
Since we are aiming for certification for headless system, graphics stack 
should not be required.
Disabling graphics by skipping JSView::CreateComponent() execution.
In the future we will remove this patch when we the graphical part will be needed.

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

Adding and apply ParagraphTxt-ComputeLineBreaks-skip-sigsegv-code.patch

ParagraphTxt::ComputeLineBreaks() is causing a segmentation fault for some 
reasons. This is causing hap applications to crash, including ACTS test suites
and status bar.
As a temporary solution we skip the part of the code that
is raising the segfault. The side effect of this is that texts are 
not properly rendered.

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

The previous implementation of HiCollieCppTest used the 'ps -A' command
to check if a process had restarted, which was not reliable.
This patch adds the creation and removal of a file
'x_collie_watchdog_test_file' in the child process that is monitored,
and the test cases now verify the presence and removal of this file to
confirm that the process was restarted.
Also, this commit updates the build recipe in
openharmony-standard_3.0.bb to include the newly created patch file.

Signed-off-by: Kareem Zarka <kareem.zarka@huawei.com>

This happens at least 60 times a second, totalling at least 120 lines per
second. We are disabling them completely, so we don't even flood the debug
log.

Signed-off-by: Esben Haabendal <esben@geanix.com>

This simplifies and improves the hack added to libdrm to handle drivers (like
virtio drm) that does not support DRM_IOCTL_WAIT_VBLANK, and tries to return at
60 Hz points in time.

Signed-off-by: Esben Haabendal <esben@geanix.com>

regenerate patch from the ${S}/base/hiviewdfx/hilog patch directory, and
apply the patch to the subdirectory.
This allows `devtool modify openharmony standard` to properly apply
the patch in the external workspace. 

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

Signed-off-by: Esben Haabendal <esben@geanix.com>

Signed-off-by: Esben Haabendal <esben@geanix.com>

The patch was a hacky way to allow starting of demo hap applications from a root user.
The proper way to start the demo apps is by running `aa start ...` from the system user.
For example:
```
$ hdc shell "su system -c 'aa start -a ohos.samples.clock.MainAbility -b ohos.samples.clock'"
```

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

remove test-xts-acts-Start-tests-from-onInit.patch because now the tests are properly
executed even without it. 
Probably after fixing a segfault problem in the graphics stack, the tests execution
have been fixed.

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

Fix DefPermission.hap filename typo which is causing a file not found error 
when running ActsContextTest ACTS tests. 
 
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

START_ABILITY_TIMEOUT is 3000ms which is too short in our case, causing some 
testcases in ActsBmsModuleUsageRecordTest to exceed timeout. 
Increasing the timeout to 30 seconds.

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

The default test timeout is 5000ms which is too short, causing some testcases
to exceed timeout. Increasing the timeout to 60 seconds.

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

OpenHarmony uses couple of UIDs/GIDs for running it's services.
Currently in meta-openharmony we have everything run as root, which 
causes some issues in starting abilities. As a temporary workaround
we are removing the Uid check.

Apply to foundation/aafwk/standard

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

RenderText::PerformLayout() is causing a SIGSEGV for some reasons. 
The error appears to be raised in the Measure method(). This is
causing hap applications to crash, including ACTS test suites.
As a temporary solution we are removing the part of the code that
is raising the segfault. The side effect of this is that texts are 
not properly rendered.

Apply patch to foundation/ace/ace_engine

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

Currently, due to some services/graphics issues onShow callback is not being
called and therefore tests are not executed. For screen-less devices graphics
stack should not be required and therefore moving test execution to onInit

This patch allows ACTS tests output to be shown on the logs

Signed-off-by: Robert Drab <robert.drab@huawei.com>
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

Robert Drab authored 2 years ago and Francesco Pham committed 2 years ago

Signed-off-by: Robert Drab <robert.drab@huawei.com>

This sets NODE_OPTIONS to pass '--openssl-legacy-provider' to node 17
and fixes the error 'digital envelope routines::unsupported'.
This was resulting in ACTS hap package not contaning the manifest and js
assets files.

Even if NODE_OPTIONS is set in the openharmony-standard recipe, it also
has to be set in the nodejs invocation environment because the recipe
environment is ignored in build_js_assets.py.

Signed-off-by: Thierry Escande <thierry.escande@huawei.com>

This test doesn't compile and is not mandatory, so remove it.

Signed-off-by: Thierry Escande <thierry.escande@huawei.com>

It is only used in two unittest testcases, and something is causing it to not be
included in the generated image, although we add the necessary parts
information.
So making it static is a simple workaround for that.

Signed-off-by: Esben Haabendal <esben.haabendal@huawei.com>

With having /system/profile a symlink to /usr/lib/openharmony/profile
and similar for /system/usr, safwk are not working, as it is using
realpath(3) on the files before comparing against dir prefix. To avoid
that, we simply change dir prefix to the resolved path.

Signed-off-by: Esben Haabendal <esben.haabendal@huawei.com>
Signed-off-by: Thierry Escande <thierry.escande@huawei.com>

This patch reworks the files for the param_service standalone
executable. It now uses the libuv signal handling APIs so it doesn't
need a libuv idle callback anymore to monitor the app exit state.

Signed-off-by: Thierry Escande <thierry.escande@huawei.com>

In OpenHarmony 3.0 Param Service is a part of the whole Init system. This
service is required by many other services. Therefore we have to split out
the service as an independent from Init when Init is not used.

Signed-off-by: Robert Drab <robert.drab@huawei.com>

Closes https://gitlab.eclipse.org/eclipse/oniro-core/meta-openharmony/-/issues/28
Closes https://gitlab.eclipse.org/eclipse/oniro-core/meta-openharmony/-/issues/31

According to OpenHarmony Device Compatibility Specification 3.0
self check sheet following tests are mandatory for screen-less
devices:

//test/xts/acts/compileruntime:compileruntime
//test/xts/acts/hiviewdfx:hiviewdfxtestacts
//test/xts/acts/startup:startup
//test/xts/acts/powermgr:powermgr
//test/xts/acts/account:account
//test/xts/acts/communication:communication
//test/xts/acts/distributeddatamgr:distributeddatamgr
//test/xts/acts/appexecfwk:appexecfwk
//test/xts/acts/aafwk:aafwk

Signed-off-by: Robert Drab <robert.drab@huawei.com>
Relates to https://gitlab.eclipse.org/eclipse/oniro-core/meta-openharmony/-/issues/24

ACTS are built for all the components used in the given target build. To be
able to compile OpenHarmony 3.0 we need to include more components than
the mandatory list specifies. As a result we have ACTS test suites built
for components we don't use. Moreover, some tests for mandatory components
are missing. This patch aligns the actual set of ACTS test suites with the
mandatory list.

Signed-off-by: Robert Drab <robert.drab@huawei.com>

There's memory corruption happening in procedure (strncpy_s call) of resetting
the longProcName which in fact is `argv`. No traces of this actually being
needed for anything could be found. Question regarding this was asked here:
https://gitee.com/openharmony/startup_appspawn/pulls/308



TODO: revisit this patch once we have more information

Signed-off-by: Robert Drab <robert.drab@huawei.com>
Relates to https://gitlab.eclipse.org/eclipse/oniro-core/meta-openharmony/-/issues/42

Signed-off-by: Robert Drab <robert.drab@huawei.com>