Resolve "API does not provide facilities to communicate job and adhoc hostnames"

This MR extends the current API by introducing two new structures
`ADM_job_resources_t` and `ADM_adhoc_resources_t`.

- [x] `ADM_adhoc_resources_t` is renamed from `ADM_storage_resources_t` 
and now contains a list of nodes assigned for the adhoc storage instance 
(the PFS is more of a static thing, so it doesn't make sense to have a 
generic interface for this). An `ADM_adhoc_resources_t` argument is added 
to `ADM_adhoc_context_create()` and is propagated through RPCs. This 
allows resource managers (i.e. Slurm or the IC) to inform us about the 
nodes assigned for an `adhoc_storage`.
- [x] A new structure `ADM_job_resources_t` is added to the API, which for 
now contains a list of nodes assigned to a job. An `ADM_job_resources_t` 
argumet is added to `ADM_register_job()` and related functions and is 
propagated to `scord` via RPCs. This allows resource managers to inform 
`scord` about the nodes assigned for a job.

Closes #59

See merge request !43

Merge branch 'amanzano/36-refactor-library-rpc-implementation-of-admire-register_adhoc_storage' into 'main'

Resolve "Refactor library RPC implementation of `admire::register_adhoc_storage`"

This MR moves the implementation of the RPC construction to `admire::detail` in `impl.[ch]pp` similarly to other RPCs already refactored. This includes changing the C ADM_types to native C++ types.

Closes #36

See merge request !26

The previous automatic ID generation caused problems with the usage of
adhoc_storage constructors outside RPC handler code (e.g. during
conversions). It makes more sense for IDs to be assigned by
a server-centric adhoc_storage_manager.

ANA MANZANO RODRIGUEZ authored 2 years ago and Alberto Miranda committed 2 years ago

ADM_register_adhoc_storage.cpp var id renamed to user_id

Resolve "Add tests to verify RPC arguments."

This MR implements tests to validate that RPC information is correctly transferred between clients and servers. To do that, we provide a `ci/check_rpcs.py` script that requires client and server logfiles with the requests to validate, plus a RPC name for the RPC of interest. The script parses the logfiles and verifies that their contents match.

In order to implement this, we allow the logging framework to be configured using environment variables, so that it is possible to easily modify the logging output file in tests. We also augment RPC replies so that they include the operation id (`op_id`) assigned by the server, so that it is possible to match client information to server information.

We also define appropriate CMake tests that run the validation script for each RPC once all the RPC tests have finished.

Closes #44 #23

See merge request !31

Closes #23

Resolve "Build fails with current Mercury `HEAD`"

Closes #65

See merge request !47

Resolve "Standing issues with `AMD_ping` logging output"

Closes #62

See merge request !45

Resolve "Issues with logging output for RPCs"

Closes #60 #61

See merge request !44