Chukwa
  1. Chukwa
  2. CHUKWA-244

should not hardcode tomcat version

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Critical Critical
    • Resolution: Duplicate
    • Affects Version/s: 0.1.2, 0.2.0
    • Fix Version/s: 0.3.0
    • Component/s: Scripts
    • Labels:
      None

      Description

      Right now, hicc.sh and start-data-processors.sh hardcode a tomcat minor version number. This is bad. As it happens, they hardcode different version numbers. This is worse.

        Activity

        Hide
        Eric Yang added a comment -

        Ari is right. This is address in CHUKWA-308. Thanks Ari.

        Show
        Eric Yang added a comment - Ari is right. This is address in CHUKWA-308 . Thanks Ari.
        Hide
        Ari Rabkin added a comment -

        Eric, do you mean CHUKWA-308? 330 hasn't been committed yet, and doesn't look applicable...

        Show
        Ari Rabkin added a comment - Eric, do you mean CHUKWA-308 ? 330 hasn't been committed yet, and doesn't look applicable...
        Hide
        Eric Yang added a comment -

        This is addressed by CHUKWA-330.

        Show
        Eric Yang added a comment - This is addressed by CHUKWA-330 .
        Hide
        Ari Rabkin added a comment -

        Is this issue now moot, given that we've gone away from using Tomcat?

        Show
        Ari Rabkin added a comment - Is this issue now moot, given that we've gone away from using Tomcat?
        Hide
        Ari Rabkin added a comment -

        I believe we should fix this before doing a release. If nothing else, by replacing the minor version number with a * in the scripts. Which is fragile, but better than status quo.

        Show
        Ari Rabkin added a comment - I believe we should fix this before doing a release. If nothing else, by replacing the minor version number with a * in the scripts. Which is fragile, but better than status quo.

          People

          • Assignee:
            Eric Yang
            Reporter:
            Ari Rabkin
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development