ENH: Add operating system field to event-tracklog

[janbjorge]

Resolves: #387

Schema will be updated as soon as we decide to publish a new schema

python -c"from pprint import pp; from fmu.dataio._metadata import generate_meta_tracklog; pp(generate_meta_tracklog())"
[{'datetime': '2024-02-12T13:46:27.159655Z',
  'event': 'created',
  'user': {'id': 'xxxxxx'},
  'sysinfo': {'fmu-dataio': {'version': '2.1.1.dev3+gbb36c6a'},
              'operating_system': {'hostname': 'AC-W6JD2YH37N',
                                   'operating_system': 'macOS-14.2.1-arm64-arm-64bit',
                                   'release': '23.2.0',
                                   'system': 'Darwin',
                                   'version': 'Darwin Kernel Version 23.2.0: '
                                              'Wed Nov 15 21:55:06 PST 2023; '
                                              'root:xnu-10002.61.3~2/RELEASE_ARM64_T6020'}}}]

[janbjorge]

sysinfo will always be present and always contain the platform field.

[perolavsvendsen]

Is platform the best word? (operating_system?) (Platform means many things, and especially in the oil industry, the word "platform" has a few additional meanings.)

How do we know that this works on all platforms? Tested so far on Mac, I assume. Those that comes to mind:

• Unix (RGS)
• Unix (LSF)
• Unix (Radix, Azure, etc)
• Windows

[perolavsvendsen]

Are there security concerns to take into account? These data are to be shared with externals, we must assume that they will be made available to anyone. Are we OK exposing exactly which platforms we are running FMU on internally?

[janbjorge]

Is platform the best word? (operating_system?) (Platform means many things, and especially in the oil industry, the word "platform" has a few additional meanings.)

How do we know that this works on all platforms? Tested so far on Mac, I assume. Those that comes to mind:

* [ ]  Unix (RGS)

* [ ]  Unix (LSF)

* [ ]  Unix (Radix, Azure, etc)

* [ ]  Windows

Its a builtin python package, i would be amazed if the does not work on all python supported platforms. However, how can i test for other platforms / operating systems?

[janbjorge]

Are there security concerns to take into account? These data are to be shared with externals, we must assume that they will be made available to anyone. Are we OK exposing exactly which platforms we are running FMU on internally?

Good question, if there is a known security weakness in a version we using this could potentially be used as a discriminator.

[jcrivenaes]

Are there security concerns to take into account? These data are to be shared with externals, we must assume that they will be made available to anyone. Are we OK exposing exactly which platforms we are running FMU on internally?

There should be people in Equinor to ask about this...

[jcrivenaes]

Is platform the best word? (operating_system?) (Platform means many things, and especially in the oil industry, the word "platform" has a few additional meanings.)
How do we know that this works on all platforms? Tested so far on Mac, I assume. Those that comes to mind:

* [ ]  Unix (RGS)

* [ ]  Unix (LSF)

* [ ]  Unix (Radix, Azure, etc)

* [ ]  Windows

Its a builtin python package, i would be amazed if the does not work on all python supported platforms. However, how can i test for other platforms / operating systems?

Yes agree it should work in general. For me, this is "nice to have" information; not something "critical" we need to make dataio/sumo work.

[jcrivenaes]

OK for me

[janbjorge]

Change since last review.

• renamed platform -> operating_system
• Added test
• Moved object initialization to _metadata.py

[janbjorge]

Is platform the best word? (operating_system?) (Platform means many things, and especially in the oil industry, the word "platform" has a few additional meanings.)
How do we know that this works on all platforms? Tested so far on Mac, I assume. Those that comes to mind:

* [ ]  Unix (RGS)

* [ ]  Unix (LSF)

* [ ]  Unix (Radix, Azure, etc)

* [ ]  Windows

Its a builtin python package, i would be amazed if the does not work on all python supported platforms. However, how can i test for other platforms / operating systems?

Yes agree it should work in general. For me, this is "nice to have" information; not something "critical" we need to make dataio/sumo work.

I checked out the source for plaform quick, and i see code for windows in there. So for now, i think its safe to assume so.

https://github.com/python/cpython/blob/3.12/Lib/platform.py

[perolavsvendsen]

I think this is fine, but we should confirm the security implications before we merge.

[perolavsvendsen]

Based on inputs from community, I consider the potential risk very low. Still a question about how much is needed to include here, but that can also be adjusted over time.