Fix opensearch-dashboard user's permission to allow reading the service configuration files

[smortex]

In #3952, the permissions where changed to fix some inconsistencies in
the .deb and .rpm packaging.

This change restricted access to the configuration files (which where
previously readable by all users) but failed to adjust the files
ownership so that the service can access these files.

Ensure the configuration directory and files belong to the root user and
the opensearch-dashboards group

Signed-off-by: Romain Tartière [email protected]

[smortex]

@peterzhuamazon I saw this trying RC .deb packages (first RC I try). It looks like it slept-through when I was working on #3952… I think it needs special labeling so that it is included in the 2.10.0 release. Thank you!

[codecov]

Codecov Report

Merging #4038 (b866ee5) into main (76479cb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #4038   +/-   ##
=======================================
  Coverage   92.10%   92.10%           
=======================================
  Files         187      187           
  Lines        5674     5674           
=======================================
  Hits         5226     5226           
  Misses        448      448           

[peterzhuamazon]

Hi @smortex ,

We do see OpenSearch rpm/deb test failed to start cluster as well.

Do you need to do the same fix on OpenSearch?

Thanks.

[smortex]

@peterzhuamazon the OpenSearch package is fine according to my testing, it already had some config that changed the owner and we kept it unchanged (just changed the permissions)

[peterzhuamazon]

@peterzhuamazon the OpenSearch package is fine according to my testing, it already had some config that changed the owner and we kept it unchanged (just changed the permissions)

Thanks @smortex I think it might be a plugin that caused some hiccups in test.
I checked in my local again with latest pkg and it runs.

I will go ahead and merge this. Thanks.