Uploaded image for project: 'Bigtop'
  1. Bigtop
  2. BIGTOP-2487

Alluxio /var/log/alluxio is not created

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.1.0
    • Fix Version/s: 1.2.0
    • Component/s: None
    • Labels:
      None

      Issue Links

        Activity

        Hide
        oflebbe Olaf Flebbe added a comment -

        Should be easy: Add directory to alluxio.dirs and add a chown -R alluxio /var/log/alluxio in the allixo.postinst in configure phase.

        Show
        oflebbe Olaf Flebbe added a comment - Should be easy: Add directory to alluxio.dirs and add a chown -R alluxio /var/log/alluxio in the allixo.postinst in configure phase.
        Hide
        githubbot ASF GitHub Bot added a comment -

        GitHub user ktsakalozos opened a pull request:

        https://github.com/apache/bigtop/pull/131

        BIGTOP-2487: Alluxio /var/log/alluxio is not created

        You can merge this pull request into a Git repository by running:

        $ git pull https://github.com/juju-solutions/bigtop BIGTOP-2487

        Alternatively you can review and apply these changes as the patch at:

        https://github.com/apache/bigtop/pull/131.patch

        To close this pull request, make a commit to your master/trunk branch
        with (at least) the following in the commit message:

        This closes #131


        commit ef4d62038e2236a6de3e99eea1cef9d948ecaf21
        Author: Konstantinos Tsakalozos <konstantinos.tsakalozos@canonical.com>
        Date: 2016-07-07T09:52:32Z

        BIGTOP-2487: Alluxio /var/log/alluxio is not created


        Show
        githubbot ASF GitHub Bot added a comment - GitHub user ktsakalozos opened a pull request: https://github.com/apache/bigtop/pull/131 BIGTOP-2487 : Alluxio /var/log/alluxio is not created You can merge this pull request into a Git repository by running: $ git pull https://github.com/juju-solutions/bigtop BIGTOP-2487 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/bigtop/pull/131.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #131 commit ef4d62038e2236a6de3e99eea1cef9d948ecaf21 Author: Konstantinos Tsakalozos <konstantinos.tsakalozos@canonical.com> Date: 2016-07-07T09:52:32Z BIGTOP-2487 : Alluxio /var/log/alluxio is not created
        Hide
        ktsakalozos Konstantinos Tsakalozos added a comment -

        alluxio is not running under alluxio user. We have to create a follow-up task for adding an alluxio user. Certainly that task would require much more testing/investigation to check that alluxio is working as expected. For now, adding the /var/log/alluxio path allows alluxio to start on Debian.

        Show
        ktsakalozos Konstantinos Tsakalozos added a comment - alluxio is not running under alluxio user. We have to create a follow-up task for adding an alluxio user. Certainly that task would require much more testing/investigation to check that alluxio is working as expected. For now, adding the /var/log/alluxio path allows alluxio to start on Debian.
        Hide
        ktsakalozos Konstantinos Tsakalozos added a comment -

        Appologies for jumpting into this, but is blocking the work on adding a charm for Alluxio. It would be great if that patch is included to 1.2.0 release so that we could have Alluxio deployable on Debian.

        Show
        ktsakalozos Konstantinos Tsakalozos added a comment - Appologies for jumpting into this, but is blocking the work on adding a charm for Alluxio. It would be great if that patch is included to 1.2.0 release so that we could have Alluxio deployable on Debian.
        Hide
        ktsakalozos Konstantinos Tsakalozos added a comment -

        Hi, is there anything I could do to increase visibility of this patch?

        Show
        ktsakalozos Konstantinos Tsakalozos added a comment - Hi, is there anything I could do to increase visibility of this patch?
        Hide
        rvs Roman Shaposhnik added a comment -

        Committed. Konstantinos Tsakalozos sorry for the delay – if you're getting stuck it is totally cool to ping us on the mailing list. That's probably the easiest way to get our attention.

        Show
        rvs Roman Shaposhnik added a comment - Committed. Konstantinos Tsakalozos sorry for the delay – if you're getting stuck it is totally cool to ping us on the mailing list. That's probably the easiest way to get our attention.
        Hide
        rvs Roman Shaposhnik added a comment -

        Konstantinos Tsakalozos also, could you please close the PR since I forgot to add the magical 'closes XXX' to the commit message.

        Show
        rvs Roman Shaposhnik added a comment - Konstantinos Tsakalozos also, could you please close the PR since I forgot to add the magical 'closes XXX' to the commit message.
        Hide
        ktsakalozos Konstantinos Tsakalozos added a comment -

        Thank you for the tip.

        Show
        ktsakalozos Konstantinos Tsakalozos added a comment - Thank you for the tip.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user ktsakalozos commented on the issue:

        https://github.com/apache/bigtop/pull/131

        Closing the PR since it got merged.

        Show
        githubbot ASF GitHub Bot added a comment - Github user ktsakalozos commented on the issue: https://github.com/apache/bigtop/pull/131 Closing the PR since it got merged.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user ktsakalozos closed the pull request at:

        https://github.com/apache/bigtop/pull/131

        Show
        githubbot ASF GitHub Bot added a comment - Github user ktsakalozos closed the pull request at: https://github.com/apache/bigtop/pull/131

          People

          • Assignee:
            ktsakalozos Konstantinos Tsakalozos
            Reporter:
            oflebbe Olaf Flebbe
          • Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development