sailfin
  1. sailfin
  2. SAILFIN-1834

Rolling Upgrade: Need some admin option to indicate use of file-based vs memory based RU procedure

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: milestone 1
    • Component/s: session_replication
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      1,834

      Description

      Memory based Rolling Upgrade should be default and there needs to be some way to
      specify use of file-based RU.

      Larry is aware of this and has a fix for it.

      • Does this also require admin changes ?

        Activity

        Hide
        lwhite added a comment -

        Fix ready - going through code review.
        No admin changes required.

        Show
        lwhite added a comment - Fix ready - going through code review. No admin changes required.
        Hide
        lwhite added a comment -

        Fix checked in. Rolling upgrade strategy now defaults to memory-based.
        It can be explicitly configured as below to file_based_rolling_upgrade
        or memory_based_rolling_upgrade as follows:

        <availability-service
        ....................
        <property name="rolling_upgrade_strategy_type"
        value="file_based_rolling_upgrade"/>
        </availability-service>

        Checking in ServerConfigLookup.java;
        /cvs/glassfish/appserv-core/src/java/com/sun/enterprise/web/ServerConfigLookup.java,v
        <-- ServerConfigLookup.java
        new revision: 1.21.2.21; previous revision: 1.21.2.20
        done
        Checking in RollingUpgradeHandler.java;
        /cvs/glassfish/appserv-core-ee/http-session-persistence/src/java/com/sun/enterprise/ee/web/sessmgmt/Attic/RollingUpgradeHandler.java,v
        <-- RollingUpgradeHandler.java
        new revision: 1.1.2.6; previous revision: 1.1.2.5
        done

        Show
        lwhite added a comment - Fix checked in. Rolling upgrade strategy now defaults to memory-based. It can be explicitly configured as below to file_based_rolling_upgrade or memory_based_rolling_upgrade as follows: <availability-service .................... <property name="rolling_upgrade_strategy_type" value="file_based_rolling_upgrade"/> </availability-service> Checking in ServerConfigLookup.java; /cvs/glassfish/appserv-core/src/java/com/sun/enterprise/web/ServerConfigLookup.java,v <-- ServerConfigLookup.java new revision: 1.21.2.21; previous revision: 1.21.2.20 done Checking in RollingUpgradeHandler.java; /cvs/glassfish/appserv-core-ee/http-session-persistence/src/java/com/sun/enterprise/ee/web/sessmgmt/Attic/RollingUpgradeHandler.java,v <-- RollingUpgradeHandler.java new revision: 1.1.2.6; previous revision: 1.1.2.5 done
        Hide
        lwhite added a comment -

        Checked same fix into SPI branch:
        Checking in ServerConfigLookup.java;
        /cvs/glassfish/appserv-core/src/java/com/sun/enterprise/web/ServerConfigLookup.java,v
        <-- ServerConfigLookup.java
        new revision: 1.21.2.20.4.1; previous revision: 1.21.2.20
        done
        Checking in RollingUpgradeHandler.java;
        /cvs/glassfish/appserv-core-ee/http-session-persistence/src/java/com/sun/enterprise/ee/web/sessmgmt/Attic/RollingUpgradeHandler.java,v
        <-- RollingUpgradeHandler.java
        new revision: 1.1.2.4.2.2; previous revision: 1.1.2.4.2.1
        done

        Show
        lwhite added a comment - Checked same fix into SPI branch: Checking in ServerConfigLookup.java; /cvs/glassfish/appserv-core/src/java/com/sun/enterprise/web/ServerConfigLookup.java,v <-- ServerConfigLookup.java new revision: 1.21.2.20.4.1; previous revision: 1.21.2.20 done Checking in RollingUpgradeHandler.java; /cvs/glassfish/appserv-core-ee/http-session-persistence/src/java/com/sun/enterprise/ee/web/sessmgmt/Attic/RollingUpgradeHandler.java,v <-- RollingUpgradeHandler.java new revision: 1.1.2.4.2.2; previous revision: 1.1.2.4.2.1 done

          People

          • Assignee:
            Scott Oaks
            Reporter:
            varunrupela
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: