Project

General

Profile

Actions

Bug #6487

closed

remove pushy from ceph-deploy

Added by Alfredo Deza over 10 years ago. Updated over 10 years ago.

Status:
Closed
Priority:
Urgent
Assignee:
Category:
ceph-deploy
Target version:
-
% Done:

0%

Source:
other
Tags:
Backport:
Regression:
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

The current state of ceph-deploy for remote connections is terrible to work with. While attempting to work on #6486 that would allow to restart monitors it was
very clear that having 3 different ways to connect and execute remote functionality is gotten out of hand.

Currently, ceph-deploy has these connections types:

  • with pushy, but using the old connection objects, no (or limited) logging, `compile()` called every where
  • with pushy, using the modified connection objects with good logging, possibility of remote hangs, and abuse of `conn.modules.some.module.call.here`
  • with remoto, using the above but in simplified ways, which is what we would end up doing

If the first 2 items do not get cleaned up, it makes very difficult and time consuming to try and add new features.

Actions #1

Updated by Alfredo Deza over 10 years ago

  • Status changed from New to In Progress
Actions #2

Updated by Alfredo Deza over 10 years ago

  • Status changed from In Progress to Closed

PR was opened: https://github.com/ceph/ceph-deploy/pull/104

And it was merged with hash: 946fc54

Actions

Also available in: Atom PDF