/srv/node referrenced before it exists

Bug #1147331 reported by Tong Li
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Object Storage (swift)
Fix Released
Undecided
Emilien Macchi

Bug Description

Instructions for a Multiple Server Swift Installation (Ubuntu) refers /srv/node directory before it exists.

Directory /srv/node should be created earlier and explained why it is needed. When you have a clean system, normally directory
/srv/node won't exist.

This is the place where /srv/node is referenced very first time in the document,

5. For every storage device in /srv/node on each node add entries to each ring:

In this paragraph, that directory should be created for swift and explain its purpose.

Changed in swift:
status: New → Triaged
Changed in swift:
assignee: nobody → Emilien Macchi (emilienm)
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to swift (master)

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

Revision history for this message
Matt Fischer (mfisch) wrote :

I think this bug is no longer valid. All the docs have moved and been completely rewritten, can we close?

Changed in swift:
status: In Progress → Invalid
status: Invalid → Incomplete
Revision history for this message
Matthew Oliver (matt-0) wrote :

In the current install guide (ubuntu), under the storage node installation: http://docs.openstack.org/juno/install-guide/install/apt/content/swift-install-storage-node.html section 3.e creates the /srv/node directory:

  # mkdir -p /srv/node/sdb1
  # mkdir -p /srv/node/sdc1

NOTE: the link we are using to point to the ubuntu installation guide is wrong, so I'll submit a patch.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to swift (master)

Reviewed: https://review.openstack.org/160196
Committed: https://git.openstack.org/cgit/openstack/swift/commit/?id=4f9a3a33427849123b12992528dd7f7dc1e6f0c6
Submitter: Jenkins
Branch: master

commit 4f9a3a33427849123b12992528dd7f7dc1e6f0c6
Author: Matthew Oliver <email address hidden>
Date: Mon Mar 2 09:21:00 2015 +1100

    Fixed links in multi-server Swift documentation

    The link to the ubuntu installation guide was incorrect and 404'ing.

    This change updates page to point (anchor) to the install guide
    section of docs.openstack.org, so it will always point to the latest
    versions of the install documentation.

    It also updates the deep links to the current install guides for the
    past few releases.

    As links continue to get outdated or we forget to add new ones, the
    anchor link at the top of the page will probably still work.

    Change-Id: I38299c03370936eb41992a953aa9d10b01a8396b
    Closes-Bug: #1147331

Changed in swift:
status: Incomplete → Fix Committed
Revision history for this message
Matthew Oliver (matt-0) wrote :

Marked as "fixed released" as the change is live on docs.o.o

Changed in swift:
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to swift (feature/ec)

Fix proposed to branch: feature/ec
Review: https://review.openstack.org/160890

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to swift (feature/ec)
Download full text (7.3 KiB)

Reviewed: https://review.openstack.org/160890
Committed: https://git.openstack.org/cgit/openstack/swift/commit/?id=dabf47c0bb6a500c0499163ba800af1f474a68bd
Submitter: Jenkins
Branch: feature/ec

commit 4f9a3a33427849123b12992528dd7f7dc1e6f0c6
Author: Matthew Oliver <email address hidden>
Date: Mon Mar 2 09:21:00 2015 +1100

    Fixed links in multi-server Swift documentation

    The link to the ubuntu installation guide was incorrect and 404'ing.

    This change updates page to point (anchor) to the install guide
    section of docs.openstack.org, so it will always point to the latest
    versions of the install documentation.

    It also updates the deep links to the current install guides for the
    past few releases.

    As links continue to get outdated or we forget to add new ones, the
    anchor link at the top of the page will probably still work.

    Change-Id: I38299c03370936eb41992a953aa9d10b01a8396b
    Closes-Bug: #1147331

commit 60cf18f5959fd91016927493097c63c9a0ab48d3
Author: OpenStack Proposal Bot <email address hidden>
Date: Fri Feb 27 06:14:07 2015 +0000

    Imported Translations from Transifex

    For more information about this automatic import see:
    https://wiki.openstack.org/wiki/Translations/Infrastructure

    Change-Id: I5032584721e390a999c84c7d4a4b0cd9c3555192

commit da6f8d8f1373fd21f2254e398eca5d989a069f66
Author: John Dickinson <email address hidden>
Date: Thu Feb 26 14:21:47 2015 -0800

    fixed ugly code pattern in probe tests

    Change-Id: I242f095ea0ca8d6d69c3b2258cce6b51c7963dce

commit 555e9894eee58fb49ac66ae703aa2835beedbe0c
Author: Clay Gerrard <email address hidden>
Date: Thu Feb 26 12:30:30 2015 -0800

    Make functests retry auth immediately

    I don't think we need to backoff after a 401 - we have a perfectly valid
    response telling us exactly what we should do so we can do that right away and
    move on.

    Assuming you run functional tests at least once a day, according to
    http://xkcd.com/1205/ and my benchmarks you can spend roughly a full day
    reviewing this change and still come out ahead.

    Change-Id: I27c42c4ee3254eb32aad4d3dac08b16b3a43d611

commit e3cbfc5c5ee47660d77ccd784e7b842539a99c1c
Author: Clay Gerrard <email address hidden>
Date: Thu Feb 26 11:45:58 2015 -0800

    Add tempauth back to /info

    Our functional tests are skipping Account ACL tests when using tempauth
    because it does not register itself with in swift.common.utils._swift_info

    Add the register_swift_info call back to the filter_factory function in the
    tempauth from which it was removed, and add a test to validate creating the
    filter will add the middleware to global info dict.

    Change-Id: I3761da3241601c93245796beb5d29bf3f36d69f6

commit a248a5c09e8fd1c110004c1e2de29ae1936e5227
Author: Mahati Chamarthy <email address hidden>
Date: Fri Feb 6 21:33:59 2015 +0530

    Ring checker in swift-recon

    This patch validates the server end points on the ring. And also generates
    a report on issues found.

    Change-Id: I913799a35d5c9178164021cfb7fcb448141b058b

commit 12e9826dbd971c...

Read more...

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to swift (feature/crypto)

Fix proposed to branch: feature/crypto
Review: https://review.openstack.org/164700

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to swift (feature/crypto)
Download full text (6.8 KiB)

Reviewed: https://review.openstack.org/164700
Committed: https://git.openstack.org/cgit/openstack/swift/commit/?id=555ff2631e84f97aceef99f9dd254621f0bcf517
Submitter: Jenkins
Branch: feature/crypto

commit b73e49e790533d668efff1290ab54f65a4048fa3
Author: Kota Tsuyuzaki <email address hidden>
Date: Mon Mar 9 03:18:25 2015 -0700

    Fix param description on _make_request

    make_requests requires a list of headers but _make_request needs
    just headers dictionary (not a list).

    Change-Id: Iaf7ba32de3702bf2189f41e42041b561c9d1b57b

commit 98911be6d72bb834f3a99f73ef6dcd66e375be43
Author: Samuel Merritt <email address hidden>
Date: Fri Mar 6 15:06:50 2015 -0800

    Small optimization to ring builder.

    We were already checking this condition a few lines up; no need to do
    it again.

    Change-Id: I066c635c8dfa3c3a1e9a944decae2f41e2c689c9

commit 7f482e9a5601b77951fef9a26166122bcdb05550
Author: Thiago da Silva <email address hidden>
Date: Fri Mar 6 14:35:40 2015 -0500

    fixing small typos in associated projects doc

    Change-Id: Ia4555acc1763a58cdf605530219ea4e4586ad517
    Signed-off-by: Thiago da Silva <email address hidden>

commit a6f630f27cc20ab4ae1cf53b6840b817aba576a0
Author: Prashanth Pai <email address hidden>
Date: Wed Oct 8 19:49:39 2014 +0530

    fsync() on directories

    renamer() method now does a fsync on containing directory of target path
    and also on parent dirs of newly created directories, by default.
    This can be explicitly turned off in cases where it is not
    necessary (For example- quarantines).

    The following article explains why this is necessary:
    http://lwn.net/Articles/457667/

    Although, it may seem like the right thing to do, this change does come
    at a performance penalty. However, no configurable option is provided to
    turn it off.

    Also, lock_path() inside invalidate_hash() was always creating part of
    object path in filesystem. Those are never fsync'd. This has been
    fixed.

    Change-Id: Id8e02f84f48370edda7fb0c46e030db3b53a71e3
    Signed-off-by: Prashanth Pai <email address hidden>

commit 4f9a3a33427849123b12992528dd7f7dc1e6f0c6
Author: Matthew Oliver <email address hidden>
Date: Mon Mar 2 09:21:00 2015 +1100

    Fixed links in multi-server Swift documentation

    The link to the ubuntu installation guide was incorrect and 404'ing.

    This change updates page to point (anchor) to the install guide
    section of docs.openstack.org, so it will always point to the latest
    versions of the install documentation.

    It also updates the deep links to the current install guides for the
    past few releases.

    As links continue to get outdated or we forget to add new ones, the
    anchor link at the top of the page will probably still work.

    Change-Id: I38299c03370936eb41992a953aa9d10b01a8396b
    Closes-Bug: #1147331

commit 13cc0efe02fffa82b4118319f069eab136f85302
Author: Alistair Coles <email address hidden>
Date: Fri Feb 27 11:54:50 2015 +0000

    Minor change to the service token sample test config

    Make the sample test config project for tester5 be test5
    to ma...

Read more...

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.