puppet master relative code in nailgun

[Igor Shishkin]

Hello,

just found the following:

in ./nailgun/nailgun/task/task.py
246 try:
247 node_db = db().query(Node).get(node['id'])
248 if node_db and node_db.fqdn:
249 node_hostname = node_db.fqdn
250 else:
251 node_hostname = TaskHelper.make_slave_fqdn(node['id'])
252 logger.info("Removing node cert from puppet: %s",
253 node_hostname)
254 cmd = "puppet cert clean {0}".format(node_hostname)

[Evgeniy L (rustyrobot)]

The problem is that we don't have puppet muster anymore, but we have hardcoded command to clean certificates from puppet master.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/74736

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/74736
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=b5eea18a35c2e971aa52a9a9190dcbe55b16e563
Submitter: Jenkins
Branch: master

commit b5eea18a35c2e971aa52a9a9190dcbe55b16e563
Author: Evgeniy L <email address hidden>
Date: Wed Feb 19 18:28:25 2014 +0400

    Remove dead code where we were removing certificates from puppet master

    Puppet master was removed from Fuel admin node
    as result we don't need to remove puppet certificates.

    Closes-bug: #1282108
    Change-Id: I2ab3f624199c5db24fb91a355e979225bb24f5fc

[Andrey Sledzinskiy (asledzinskiy)]

verified