#2367 closed defect (fixed)

manhole should be either tested or removed

Reported by: daira Owned by: daira
Priority: normal Milestone: 1.12.0
Component: code Version: 1.10.0
Keywords: tests manhole ssh Cc:
Launchpad Bug:

Description


Change History (3)

comment:1 Changed at 2015-02-05T16:39:38Z by daira

  • Summary changed from add tests for manhole to manhole should be either tested or removed

comment:2 Changed at 2016-04-07T00:01:04Z by Brian Warner <warner@…>

  • Resolution set to fixed
  • Status changed from new to closed

In 8279d91/trunk:

remove "manhole" (ssh-accessible REPL)

This little-used debugging feature allowed you to SSH or Telnet "into" a
Tahoe node, and get an interactive Read-Eval-Print-Loop (REPL) that
executed inside the context of the running process. The SSH
authentication code used a deprecated feature of Twisted, this code had
no unit-test coverage, and I haven't personally used it in at least 6
years (despite writing it in the first place). Time to go.

Also experiment with a Twisted-style "topfiles/" directory of NEWS
fragments. The idea is that we require all user-visible changes to
include a file or two (named as $TICKETNUM.$TYPE), and then run a script
to generate NEWS during the release process, instead of having a human
scan the commit logs and summarize the changes long after they landed.

Closes ticket:2367

comment:3 Changed at 2016-04-07T00:01:47Z by warner

  • Milestone changed from undecided to 1.12.0
Note: See TracTickets for help on using tickets.