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

package deployment tests are CDH specific: _has_ to be removed or fixed

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 0.1.0
    • Fix Version/s: 1.2.0
    • Component/s: tests
    • Labels:

      Description

      Test DeployCDH seems to be completely irrelevant for Bigtop purposes, unless it is properly reflects Bigtop packages structure.

      1. 0001-BIGTOP-976.-package-deployment-tests-are-CDH-specifi.patch
        12 kB
        Konstantin Boudnik
      2. BIGTOP-976.patch
        9 kB
        Konstantin Boudnik

        Issue Links

          Activity

          Hide
          cos Konstantin Boudnik added a comment -

          Pushed to the master

          Show
          cos Konstantin Boudnik added a comment - Pushed to the master
          Hide
          cos Konstantin Boudnik added a comment -

          The original patch bit-rotted over time. Here's the updated one, removing yet another vendor specific file. I will commit it shortly as it is effectively the same as before.

          Show
          cos Konstantin Boudnik added a comment - The original patch bit-rotted over time. Here's the updated one, removing yet another vendor specific file. I will commit it shortly as it is effectively the same as before.
          Hide
          cos Konstantin Boudnik added a comment -

          No reason, just was wondering if they can help to identify other places (which I clearly don't remember considering that like 6 years have passed). Considering the request was here for almost two years, it'd be silly to expect anyone will really chime in. I will commit this. Thanks for looking, Evans!

          Show
          cos Konstantin Boudnik added a comment - No reason, just was wondering if they can help to identify other places (which I clearly don't remember considering that like 6 years have passed). Considering the request was here for almost two years, it'd be silly to expect anyone will really chime in. I will commit this. Thanks for looking, Evans!
          Hide
          evans_ye Evans Ye added a comment -

          The patch looks good. I think we should remove them directly.
          Any reason requires CDH folks to chime in?
          Otherwise, +1.

          Show
          evans_ye Evans Ye added a comment - The patch looks good. I think we should remove them directly. Any reason requires CDH folks to chime in? Otherwise, +1.
          Hide
          cos Konstantin Boudnik added a comment -

          Can anyone from Cloudera folks look at it?

          Show
          cos Konstantin Boudnik added a comment - Can anyone from Cloudera folks look at it?
          Hide
          cos Konstantin Boudnik added a comment -

          Initial patch... more to come.

          Show
          cos Konstantin Boudnik added a comment - Initial patch... more to come.
          Hide
          rvs Roman Shaposhnik added a comment -

          I fully agree it needs to be removed. Care to submit a patch?

          Show
          rvs Roman Shaposhnik added a comment - I fully agree it needs to be removed. Care to submit a patch?
          Hide
          cos Konstantin Boudnik added a comment -

          The class is used by TestPackagesPseudoDistributedUpgrade suite, which doesn't seem to be currently running at all.

          Show
          cos Konstantin Boudnik added a comment - The class is used by TestPackagesPseudoDistributedUpgrade suite, which doesn't seem to be currently running at all.

            People

            • Assignee:
              cos Konstantin Boudnik
              Reporter:
              cos Konstantin Boudnik
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development