Alon Bar-Lev has posted comments on this change. Change subject: packaging: initial repo creation ......................................................................
Patch Set 18: (12 inline comments) .................................................... File build.sh Line 1: #!/usr/bin/sh Line 2: yum install -y spin-kickstarts.noarch Line 3: yum install -y livecd-tools Line 4: cp -f /usr/share/spin-kickstarts/fedora-live-base.ks ./kickstart/ovirt-live-base.ks Line 5: sed -i 's/selinux \-\-enforcing/selinux \-\-permissive/g' ./kickstart/ovirt-live-base.ks please provide these seds as patch. Line 6: sed -i 's/services \-\-enabled\=NetworkManager \-\-disabled\=network\,sshd/services \-\-enabled\=NetworkManager\,sshd \-\-disabled\=network\,firewalld\nrepo \-\-name\=ovirt \-\-baseurl\=http:\/\/ovirt.org\/releases\/nightly\/rpm\/Fedora\/18/g' ./kickstart/ovirt-live-base.ks Line 7: sed -i 's/liveuser/oVirtuser/g' ./kickstart/ovirt-live-base.ks Line 8: sed -i 's/firewall \-\-enabled/firewall \-\-disabled/g' ./kickstart/ovirt-live-base.ks Line 9: sed -i 's/livedir\=\"LiveOS\"/livedir\=\"LiveoVirt\"/g' ./kickstart/ovirt-live-base.ks .................................................... File kickstart/ovirt.ks Line 72: mkdir /root/oVirtLiveFiles/ Line 73: echo "making autostart dir" Line 74: mkdir -p ~oVirtuser/.config/autostart Line 75: Line 76: cat >> /etc/rc.d/init.d/livesys << EOF can we put this as external file? Line 77: Line 78: # make the installer show up Line 79: if [ -f /usr/share/applications/liveinst.desktop ]; then Line 80: # Show harddisk install in shell dash Line 75: Line 76: cat >> /etc/rc.d/init.d/livesys << EOF Line 77: Line 78: # make the installer show up Line 79: if [ -f /usr/share/applications/liveinst.desktop ]; then this too? Line 80: # Show harddisk install in shell dash Line 81: sed -i -e 's/NoDisplay=true/NoDisplay=false/' /usr/share/applications/liveinst.desktop "" Line 82: # need to move it to anaconda.desktop to make shell happy Line 83: mv /usr/share/applications/liveinst.desktop /usr/share/applications/anaconda.desktop Line 81: sed -i -e 's/NoDisplay=true/NoDisplay=false/' /usr/share/applications/liveinst.desktop "" Line 82: # need to move it to anaconda.desktop to make shell happy Line 83: mv /usr/share/applications/liveinst.desktop /usr/share/applications/anaconda.desktop Line 84: Line 85: cat >> /usr/share/glib-2.0/schemas/org.gnome.shell.gschema.override << FOE this too? Line 86: [org.gnome.shell] Line 87: favorite-apps=['ovirt.desktop' , 'engine-setup.desktop','firefox.desktop', 'evolution.desktop', 'empathy.desktop', 'shotwell.desktop', 'nautilus.desktop', 'anaconda.desktop'] Line 88: FOE Line 89: Line 111: Line 112: # Add oVirtuser to sudoers Line 113: /sbin/usermod -g wheel oVirtuser Line 114: Line 115: sed -i 's/utils.retry(isHostUp, tries=20, timeout=150, sleep=5)/utils.retry(isHostUp, tries=100, timeout=600, sleep=5)/g' /usr/share/ovirt-engine/scripts/plugins/all_in_one_100.py better to allow all_in_one to accept extra parameters than to hack it, it is our script after all. Line 116: # No reboot is needed in AIO, so manager should not wait Line 117: sed -i "s/'ServerRebootTimeout','300'/'ServerRebootTimeout','0'/g" /usr/share/ovirt-engine/dbscripts/upgrade/pre_upgrade/0000_config.sql Line 118: # Allow wlan* on host (as we're running locally) Line 119: sed -i -e 's/# hidden_nics = wlan\*,usb\*/hidden_nics = usb\*/' /etc/vdsm/vdsm.conf Line 113: /sbin/usermod -g wheel oVirtuser Line 114: Line 115: sed -i 's/utils.retry(isHostUp, tries=20, timeout=150, sleep=5)/utils.retry(isHostUp, tries=100, timeout=600, sleep=5)/g' /usr/share/ovirt-engine/scripts/plugins/all_in_one_100.py Line 116: # No reboot is needed in AIO, so manager should not wait Line 117: sed -i "s/'ServerRebootTimeout','300'/'ServerRebootTimeout','0'/g" /usr/share/ovirt-engine/dbscripts/upgrade/pre_upgrade/0000_config.sql this should be done using engine-config -s ServerRebootTimeout=300 Line 118: # Allow wlan* on host (as we're running locally) Line 119: sed -i -e 's/# hidden_nics = wlan\*,usb\*/hidden_nics = usb\*/' /etc/vdsm/vdsm.conf Line 120: sed -i 's/\[vars\]/\[vars\]\nfake_nics\ \=\ \dummy0/g' /etc/vdsm/vdsm.conf Line 121: %end Line 116: # No reboot is needed in AIO, so manager should not wait Line 117: sed -i "s/'ServerRebootTimeout','300'/'ServerRebootTimeout','0'/g" /usr/share/ovirt-engine/dbscripts/upgrade/pre_upgrade/0000_config.sql Line 118: # Allow wlan* on host (as we're running locally) Line 119: sed -i -e 's/# hidden_nics = wlan\*,usb\*/hidden_nics = usb\*/' /etc/vdsm/vdsm.conf Line 120: sed -i 's/\[vars\]/\[vars\]\nfake_nics\ \=\ \dummy0/g' /etc/vdsm/vdsm.conf I think we can provide our own vdsm.conf and not edit in-place. Line 121: %end Line 122: Line 123: %post --nochroot Line 124: cp -r oVirtLiveFiles $INSTALL_ROOT/root/ Line 143: #configuring autologin Line 144: sed -i 's/\[daemon\]/\[daemon\]\nAutomaticLoginEnable\=True\nAutomaticLogin\=oVirtuser/g' /etc/gdm/custom.conf Line 145: Line 146: #settings up vdsm to use a dummy nic Line 147: cp /home/oVirtuser/oVirtLiveFiles/50-vdsm-conf-fake-nic.conf /etc/ovirt-host-deploy.conf.d/ you can create a link? Anyway, installing at /home/ is not wise as the permissions of /home are unique to user, best to install files at /usr/share/ Line 148: echo '10.0.0.1 ovirtlive.localdomain localdomain' >> /etc/hosts Line 149: Line 150: #setting wallapaper Line 151: cp /home/oVirtuser/oVirtLiveFiles/images/ovirt-wallpaper-16:9.jpg /usr/share/backgrounds/spherical-cow/default/standard/spherical-cow.png Line 149: Line 150: #setting wallapaper Line 151: cp /home/oVirtuser/oVirtLiveFiles/images/ovirt-wallpaper-16:9.jpg /usr/share/backgrounds/spherical-cow/default/standard/spherical-cow.png Line 152: cp /home/oVirtuser/oVirtLiveFiles/images/ovirt-wallpaper-16:9.jpg /usr/share/backgrounds/spherical-cow/default/wide/spherical-cow.png Line 153: cp /home/oVirtuser/oVirtLiveFiles/images/ovirt-wallpaper-16:9.jpg /usr/share/backgrounds/spherical-cow/default/normalish/spherical-cow.png I expect something like: cp -r /usr/share/ovirt-live/files/* / Line 154: Line 155: #copying network files Line 156: cp /home/oVirtuser/oVirtLiveFiles/etc/sysconfig/network-scripts/* /etc/sysconfig/network-scripts/ Line 157: Line 164: #setting up autologin Line 165: cp /home/oVirtuser/oVirtLiveFiles/etc/gdm/* /etc/gdm/ Line 166: Line 167: #overriding glib schemas Line 168: cp /home/oVirtuser/oVirtLiveFiles/usr/share/glib-2.0/schemas/* /usr/share/glib-2.0/schemas/ it solves all the above. Line 169: Line 170: glib-compile-schemas /usr/share/glib-2.0/schemas Line 171: Line 172: #workaround for bz 878119 Line 176: yum localinstall -y /home/oVirtuser/oVirtLiveFiles/rpms/*.rpm Line 177: Line 178: #updating patched files Line 179: cp /home/oVirtuser/oVirtLiveFiles/ovirt_live_101.py /usr/share/ovirt-engine/scripts/plugins/ Line 180: cp /home/oVirtuser/oVirtLiveFiles/ovirt_live_101.py /usr/share/ovirt-engine/scripts/ same as above. Line 181: Line 182: Line 183: # Manipulate fqdn validation, so that it is possible to setup with answer file Line 184: sed -i 's/raise Exception(output_messages.ERR_EXP_VALIDATE_PARAM % param.getKey("CONF_NAME"))/logging.debug("Failed to validate %s with value %s",param,paramValue)/g' /usr/share/ovirt-engine/scripts/engine-setup.py Line 180: cp /home/oVirtuser/oVirtLiveFiles/ovirt_live_101.py /usr/share/ovirt-engine/scripts/ Line 181: Line 182: Line 183: # Manipulate fqdn validation, so that it is possible to setup with answer file Line 184: sed -i 's/raise Exception(output_messages.ERR_EXP_VALIDATE_PARAM % param.getKey("CONF_NAME"))/logging.debug("Failed to validate %s with value %s",param,paramValue)/g' /usr/share/ovirt-engine/scripts/engine-setup.py patch please Line 185: sed -i -e 's/\(^SELINUX=\).*$/\1permissive/' /etc/selinux/config Line 186: #yad script - give some gui to installation Line 187: Line 188: echo "finishing post section" -- To view, visit http://gerrit.ovirt.org/9733 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I66cca8f5d5abaa710f054c59043ff6363b68256f Gerrit-PatchSet: 18 Gerrit-Project: ovirt-live Gerrit-Branch: master Gerrit-Owner: Moran Goldboim <mgold...@redhat.com> Gerrit-Reviewer: Alex Lourie <alou...@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Eyal Edri <ee...@redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com> Gerrit-Reviewer: Kiril Nesenko <knese...@redhat.com> Gerrit-Reviewer: Moran Goldboim <mgold...@redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschr...@redhat.com> Gerrit-Reviewer: Ohad Basan <oba...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches