Adds logging if Pulp is missing critical Celery services #1976
Adds logging if Pulp is missing critical Celery services #1976
Conversation
Refer to this link for build results (access rights to CI server needed): |
elif worker['name'].startswith(SCHEDULER_WORKER_NAME): | ||
scheduler_count = scheduler_count + 1 | ||
elif worker['name'].startswith(RESOURCE_MANAGER_WORKER_NAME): | ||
resource_manager_count = resource_manager_count + 1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It may be worthwhile to add a debug-level logging statement here for total number of schedulers and resource managers.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a good idea; I'm doing it and an associated test.
LGTM but I would not mind a second review since it's for 2.6.x 🐨 |
""" | ||
msg = _('Looking for workers missing for more than %s seconds') % self.WORKER_TIMEOUT_SECONDS | ||
msg = _('Checking if pulp_workers, pulp_celerybeat, or pulp_resource_manager ' | ||
'processes are missing for more than %s seconds') % self.CELERY_TIMEOUT_SECONDS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
python is very forgiving, but in theory this should be %d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good catch; fixed
b7072f3
to
21eaaea
Compare
Refer to this link for build results (access rights to CI server needed): |
Adds logging if Pulp is missing critical Celery services
…nested directories Solution: Enable RSyncFastForwardUnitPublishStep to publish the same type of units into different directories by extracting the publish path of a unit from it's symlink location. re pulp#1976 https://pulp.plan.io/issues/1976
Solution: Enable RSyncFastForwardUnitPublishStep to publish the same type of units into different directories by extracting the publish path of a unit from it's symlink location. This problem was introduced when the yum distributor was modified to publish repositories with a new layout. This patch makes the RPM rsync distributor consistent with the yum distributor. re pulp#1976 https://pulp.plan.io/issues/1976
Solution: Enable RSyncFastForwardUnitPublishStep to publish the same type of units into different directories by extracting the publish path of a unit from it's symlink location. This problem was introduced when the yum distributor was modified to publish repositories with a new layout. This patch makes the RPM rsync distributor consistent with the yum distributor. re pulp#1976 https://pulp.plan.io/issues/1976
https://pulp.plan.io/issues/1115
closes 1115