Refactor: libpe_status: create_container() -> create_replica_resources()

The new name is more accurate and is less confusing since we now have a
create_container_resource() function.

Also do some minor cleanup of create_replica_resources() and its called
functions. Get rid of an unnecessary data_set argument and make the rest
of the functions return a standard Paceamker return code.
(create_container_resource() already did this.)

This isn't comprehensive. It leaves a lot of best practices unaddressed
and probably leaves some potential const arguments non-const.

Signed-off-by: Reid Wahl <nrwahl@protonmail.com>