-
Notifications
You must be signed in to change notification settings - Fork 697
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Photon 3.0 r2 pxe boot install fail repo= not handled properly #953
Comments
@kl1059, not sure what you mean. As it stands, the repo path passed to isoInstaller.py takes precedence over that sent over kernel command line. |
In Photon 3.0 Rev 2 we introduced command line parameter '--repo-path' for isoInstaller.py. If '--repo-path' is not set, then kernel's 'repo=' will be used. What kind of installation failure are you facing? Can you attach screenshot or /var/log/installer.log file, please? |
PXE boot install of photon-3.0 succeeds PXE boot install of photon-3.0 rev 2 fails just after answering Yes to the pxe cfg entries: LABEL Photon Linux 3.0 |
From the log file I see 'repo' was set to /mnt/cdrom/RPMS. Which seems incorrect. At any time during the installation, press Ctrl+C, or pick "Cancel" at Welcome screen to get to shell,
|
the change to /bin/bootphotoninstaller works! we will make a new initrd with this change. thanks. this did, however, raise another issue which would only be seen by a pxe boot with an http repo. the mode of the rpm files on the 3.0 CD are 644. most of the rpm files on the 3.0 rev 2 CD have the execute bit lit and some http servers do not like to serve executeable files if they are not explicity declared as cgi. it is a minor nitpick and we can change the mode of files on the http server easily. |
Glad that it worked. We are fixing it right now. |
updated 3.0 r2 bits. |
Photon 3.0 pxe boot installs with repo=http://192.168.1.1/RPMS/ fine
./installer/iso_config.py detects repo= in the kernel args and sets rpm_path accordingly
In Photon 3.0 Rev 2 pxe boot installs fail, repo= is no longer correctly handled
in ./installer/isoInstaller.py repo_path = options.repo_path which may not be null
later "if not repo_path: repo_path = arg[len("repo="):]
perhaps the "if not repo_path:" at line 32 should be removed?
The text was updated successfully, but these errors were encountered: