#1477 closed enhancement (fixed)

reformatting and minor corrections to webapi.rst; remove a reference to the test grid

Reported by: davidsarah Owned by: daira
Priority: minor Milestone: 1.10.0
Component: documentation Version: 1.8.2
Keywords: docs webapi Cc:
Launchpad Bug:

Description

Reflow, change tabs to spaces, and make some minor corrections (see the third patch in the patchset).

Attachments (1)

edit-webapi-doc.darcs.patch (72.0 KB) - added by davidsarah at 2011-08-11T01:16:10Z.
docs/frontends/webapi.txt: clean up formatting (reflo, tabs->spaces); minor corrections; remove reference to test grid

Download all attachments as: .zip

Change History (14)

Changed at 2011-08-11T01:16:10Z by davidsarah

docs/frontends/webapi.txt: clean up formatting (reflo, tabs->spaces); minor corrections; remove reference to test grid

comment:1 Changed at 2011-08-11T01:18:19Z by davidsarah

  • Keywords review-needed added
  • Owner changed from somebody to zooko

comment:2 Changed at 2011-08-16T20:54:11Z by zooko

  • Status changed from new to assigned

comment:3 Changed at 2012-04-01T04:50:56Z by davidsarah

  • Milestone changed from soon to 1.10.0

comment:4 Changed at 2012-04-01T22:46:43Z by davidsarah

  • Keywords review-needed removed
  • Owner changed from zooko to davidsarah
  • Status changed from assigned to new

This needs rebasing.

comment:5 Changed at 2013-03-15T02:56:05Z by marlowe

DS,

Would it be helpful for me to do the rebasing for this ticket?

Patrick

comment:6 Changed at 2013-03-21T00:09:52Z by warner

marlowe: yeah, that'd be great. I'd suggest just walking through davidsarah's diff in one window and applying the suggested changes by hand to the current trunk, then looking around for other instances of the same sorts of changes (i.e. search for tabs and fix them all). I wouldn't be surprised if the original diff doesn't apply cleanly after all this time: it calls for a human touch.

thanks!

comment:7 Changed at 2013-03-23T02:38:47Z by davidsarah

  • Owner changed from davidsarah to marlowe

Please do, thanks.

comment:8 Changed at 2013-03-26T05:41:52Z by davidsarah

Could you also please apply this patch:

[[1.9.2 branch] Add comments and a caveat in webapi.rst indicating that
the needs-rebalancing field may be computed incorrectly. refs #1115 refs
#1784
david-sarah@jacaranda.org**20120701230843
 Ignore-this: fd2939ae4291207978dc463a2ad92d1b
] {
hunk ./docs/frontends/webapi.rst 1350
-                            shares
+                            shares. Note that a high value does not
necessarily
+                            imply good share distribution, because some of
+                            these servers may only hold duplicate shares.
hunk ./docs/frontends/webapi.rst 1367
-    needs-rebalancing: (bool) True if there are multiple shares on a single
-                       storage server, indicating a reduction in
reliability
-                       that could be resolved by moving shares to new
-                       servers.
+    needs-rebalancing: (bool) This field is intended to be True iff
+                       reliability could be improved for this file by
+                       rebalancing, i.e. by moving some shares to other
+                       servers. It is not guaranteed to be computed
correctly
+                       in Tahoe-LAFS up to and including v1.9.2, and its
+                       precise definition may change in future versions.
hunk ./src/allmydata/immutable/checker.py 799
+        # TODO: this may be wrong, see ticket #1115 comment:27 and
ticket #1784.
hunk ./src/allmydata/immutable/filenode.py 134
+
+                    # TODO: this may be wrong, see ticket #1115
comment:27 and ticket #1784.
}

Thanks.

comment:9 Changed at 2013-03-28T03:47:27Z by daira

Just the part of the patch that applies to webapi.rst, I meant.

comment:10 Changed at 2013-04-09T19:03:51Z by zooko

Since this ticket was opened, the test grid has been revived by PRabahy. Does that mean this doc should be adjusted to once again refer to the test grid?

Whatever we do, we should do it soon! Brian will be making an RC1 release of Tahoe-LAFS v1.10.0 soon, and it would be good for these docs to go into that so people can test the docs.

comment:11 Changed at 2013-04-18T22:16:35Z by daira

  • Owner changed from marlowe to daira
  • Status changed from new to assigned

comment:12 Changed at 2013-04-18T22:50:13Z by Daira Hopwood <david-sarah@…>

In b06f8cd8d03a6239:

Add comments and a caveat in webapi.rst indicating that
the needs-rebalancing field may be computed incorrectly. refs #1115, #1784, #1477

Signed-off-by: Daira Hopwood <david-sarah@…>

comment:13 Changed at 2013-04-18T22:52:34Z by daira

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

Also in 6899d028540fbd4f. I just dropped the formatting changes; I don't think they're important.

Last edited at 2013-04-18T22:52:56Z by daira (previous) (diff)
Note: See TracTickets for help on using tickets.