(#16570) Don't load the node object again in configurer

[dalen]

Avoid loading the node object again if we already have a catalog.

In the long run the configurer class should probably be rafactored to have separate methods for fetching a catalog and applying it (and possibly have fetching of catalogs elsewhere).

Not methods like this:

# Retrieve (optionally) and apply a catalog. If a catalog is passed in
# the options, then apply that one, otherwise retrieve it.
def apply_catalog(catalog, options)

[hlindberg]

Not sure if it is applicable (depends on where the catalog comes from)... did you consider what happens if a node is changed from one environment to another and it has a catalog compiled for the "old" environment. (Not sure if that can happen at all and get to this point).

[pcarlisle]

If a catalog already exists at this point it's been intentionally supplied by the user.

[puppetcla]

CLA signed by all contributors.