Whirr
  1. Whirr
  2. WHIRR-179

[Hadoop] Guard $MOUNT/tmp mkdir call against existing directory

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.3.0
    • Fix Version/s: 0.3.0
    • Component/s: None
    • Labels:
      None

      Description

      Again, this is caused by other services running on the same machines trying to create the same directory. Simply add a check before calling mkdir.

      1. WHIRR-179.patch
        0.6 kB
        Lars George
      2. WHIRR-179-2.patch
        1 kB
        Lars George

        Activity

        Hide
        Lars George added a comment -

        Adds check as mentioned.

        Show
        Lars George added a comment - Adds check as mentioned.
        Hide
        Tom White added a comment -

        Does scripts/apache/hbase/post-configure need this check too?

        Show
        Tom White added a comment - Does scripts/apache/hbase/post-configure need this check too?
        Hide
        Lars George added a comment -

        It has https://github.com/larsgeorge/whirr/commit/0e713f63fb0cea6718ea84bd65a2e03887bdc02c but I was waiting for you to accept or cancel the WHIRR-25 patch and then rebase these other changes and up them as new issues accordingly. Makes sense?

        Show
        Lars George added a comment - It has https://github.com/larsgeorge/whirr/commit/0e713f63fb0cea6718ea84bd65a2e03887bdc02c but I was waiting for you to accept or cancel the WHIRR-25 patch and then rebase these other changes and up them as new issues accordingly. Makes sense?
        Hide
        Tom White added a comment -

        Yes. Sorry about the delay in getting WHIRR-25 in. Hopefully you can rebase against trunk now.

        Show
        Tom White added a comment - Yes. Sorry about the delay in getting WHIRR-25 in. Hopefully you can rebase against trunk now.
        Hide
        Lars George added a comment -

        v2 of the patch adding the same guard for HBase service.

        Show
        Lars George added a comment - v2 of the patch adding the same guard for HBase service.
        Hide
        Tom White added a comment -

        I've just committed this. Thanks Lars!

        Show
        Tom White added a comment - I've just committed this. Thanks Lars!

          People

          • Assignee:
            Lars George
            Reporter:
            Lars George
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development