Frank Kobzik has posted comments on this change.
Change subject: engine: GMT timezone for linux vms
......................................................................
Patch Set 2: (4 inline comments)
I'm going to push the new patchset in a moment.
....................................................
File backend/manager/dbscripts/upgrade/03_03_0210_upgrade_utcdiff_for_linux.sql
Line 5: FROM vm_static AS vs
Line 6: WHERE vd.vm_guid = vs.vm_guid AND vs.os IN(5, 7, 8, 9, 13, 14, 15, 18,
19);
Line 7:
Line 8: UPDATE vm_static
Line 9: SET time_zone = ''
Well, I think it should be NULL or '' - it's the default value (when you create
a new VM it's null). And if it's default then in VmInfoBuilderBase the
"Etc/GMT" timezone is used . In the end it doesn't really matter.
....................................................
File
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VM.java
Line 1156: setAcpiEnable(vm.getAcpiEnable());
Line 1157: setGuestCurrentUserName(vm.getGuestCurrentUserName());
Line 1158: setWin2kHackEnable(vm.getWin2kHackEnable());
Line 1159: if (getOs() != null && !getOs().isLinux()) {
Line 1160: setUtcDiff(vm.getUtcDiff());
But what about updates from different places (db)? I was told that there are
some users who want nonzero offset. This should only forbid updates from vdsm
to prevent accumulative drift error (caused by qemu or libvirt).
Line 1161: }
Line 1162: setExitStatus(vm.getExitStatus());
Line 1163: setExitMessage(vm.getExitMessage());
Line 1164: setClientIp(vm.getClientIp());
....................................................
File
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java
Line 175:
getModel().getTimeZone().setSelectedItem(Linq.firstOrDefault(timeZones, new
Linq.TimeZonePredicate(selectedTimeZone)));
Line 176:
Line 177: if (getModel().getIsLinuxOS()) { // For linux disable
editing
Line 178: getModel().getTimeZone().setIsChangable(false);
Line 179:
getModel().getTimeZone().setTitle(constants.timeZoneNotEditableForLinuxVms());
// $NON-NLS-1$
Done
Line 180: } else {
Line 181: getModel().getTimeZone().setIsChangable(true);
Line 182: getModel().getTimeZone().setTitle(""); //
$NON-NLS-1$
Line 183: }
....................................................
File
frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java
Line 1891: // Gluster Hook
Line 1892: @DefaultStringValue("Please select a resolve action to continue")
Line 1893: String noResolveActionSelectedGlusterHook();
Line 1894:
Line 1895: @DefaultStringValue("Time Zone is not editable for Linux VMs")
I'm not sure about this one. "Cannot change timezone..." just evokes a feeling
there is something wrong with the machine. Maybe I'll put "Time Zone is not
changeable..".
Line 1896: String timeZoneNotEditableForLinuxVms();
--
To view, visit http://gerrit.ovirt.org/15802
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: If681a23a450a22685c7110a337b08f3c67609c34
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Frank Kobzik <[email protected]>
Gerrit-Reviewer: Aaron Bawcom <[email protected]>
Gerrit-Reviewer: Arik Hadas <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Frank Kobzik <[email protected]>
Gerrit-Reviewer: Martin Beták <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: Roy Golan <[email protected]>
Gerrit-Reviewer: Tomas Jelinek <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
