Skip to content
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.

Sign up for GitHub

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

Jump to bottom

aii pxelinux write failure #258

Closed
stdweird opened this issue Mar 2, 2017 · 7 comments
Closed

aii pxelinux write failure #258

stdweird opened this issue Mar 2, 2017 · 7 comments
Assignees
@jouvin
Milestone
17.3

Comments

@stdweird
Copy link
Member

stdweird commented Mar 2, 2017

related to #225

ugly failure due to missing function/method?

[ERROR] Failed to execute plugin module's pxelinux Configure method
[ERROR] Errors running plugin module pxelinux Configure method: Undefined subroutine &NCM::Component::pxelinux::write_pxelinux_config called at /usr/lib/perl/NCM/Component/pxelinux.pm line 720.

[INFO] ran configure on 1 nodes
[INFO] install step for reinstall option.
[INFO] Detected domainname within configfile RPC section: [banette.os]
[INFO] Detected OpenNebula version: 5.2.1
[ERROR] VM template is not available to instantiate VM: test2801.banette.os
[INFO] ran install on 1 nodes
Uncaught exception!!! Calling stack is:
  LC::Fatal::symlink called at /usr/lib/perl/NCM/Component/pxelinux.pm line 578
  NCM::Component::pxelinux::__ANON__ called at /usr/lib/perl/AII/Shellfe.pm line 494
  AII::Shellfe::__ANON__ called at /usr/lib/perl/AII/Shellfe.pm line 1087
  AII::Shellfe::cmds called at /usr/sbin/aii-shellfe line 352
*** symlink(test2801.banette.os.cfg, /osinstall/nbp/grub-efi/grub.cfg-0A8D15C9): No such file or directory
@stdweird stdweird added this to the 17.3 milestone Mar 2, 2017
@stdweird
Copy link
Member Author

stdweird commented Mar 2, 2017

@jouvin

@stdweird
Copy link
Member Author

stdweird commented Mar 2, 2017

@jouvin this is during a reinstall, no uefi settings applied yet

@stdweird
Copy link
Member Author

stdweird commented Mar 2, 2017

fixed with comments in PR https://github.com/quattor/aii/pull/225/files#r103927865 and https://github.com/quattor/aii/pull/225/files#r103927717

@jouvin
Copy link
Contributor

jouvin commented Mar 2, 2017

I apologize about this. I did this yesterday evening and I was sure that I installed the new version since then... Clearly not. And I was sure that a typo would be catched by unit tests when these methods are not. PR #255 fixed now. Unit test still to be added: I keep this issue open until it is done.

jouvin added a commit to jouvin/aii that referenced this issue Mar 2, 2017
@jouvin
pxelinux: add kernel parameters in Grub2 config
e827956 
- Includes some code refactoring/cleanup
- Unit tests adjusted to test PXELINUX and Grub2 variants, including
  - pxelinux_no_append_block.t removed (superseded by write_pxelinux_config.t)
  - Several pxelinux_ks_xxx tests merged for easier maintenance (coverage
    and regexps tests are identical)
- Fixes quattor#258 (caused by a typo in the initial version of this commit)
jouvin added a commit to jouvin/aii that referenced this issue Mar 2, 2017
@jouvin
pxelinux: add kernel parameters in Grub2 config
c302c98 
- Includes some code refactoring/cleanup
- Unit tests adjusted to test PXELINUX and Grub2 variants, including
  - pxelinux_no_append_block.t removed (superseded by write_pxelinux_config.t)
  - Several pxelinux_ks_xxx tests merged for easier maintenance (coverage
    and regexps tests are identical)
- Fixes quattor#258 (caused by a typo in the initial version of this commit)
jouvin added a commit to jouvin/aii that referenced this issue Mar 4, 2017
@jouvin
pxelinux: add kernel parameters in Grub2 config
834f16f 
- Includes some code refactoring/cleanup
- Unit tests adjusted to test PXELINUX and Grub2 variants, including
  - pxelinux_no_append_block.t removed (superseded by write_pxelinux_config.t)
  - Several pxelinux_ks_xxx tests merged for easier maintenance (coverage
    and regexps tests are identical)
- Fixes quattor#258 (caused by a typo in the initial version of this commit)
jouvin added a commit to jouvin/aii that referenced this issue Mar 8, 2017
@jouvin
pxelinux: add kernel parameters in Grub2 config
d83621e 
- Includes some code refactoring/cleanup
- Unit tests adjusted to test PXELINUX and Grub2 variants, including
  - pxelinux_no_append_block.t removed (superseded by write_pxelinux_config.t)
  - Several pxelinux_ks_xxx tests merged for easier maintenance (coverage
    and regexps tests are identical)
- Fixes quattor#258 (caused by a typo in the initial version of this commit)
jouvin added a commit to jouvin/aii that referenced this issue Mar 19, 2017
@jouvin
pxelinux: add kernel parameters in Grub2 config
cd85854 
- Includes some code refactoring/cleanup
- Unit tests adjusted to test PXELINUX and Grub2 variants, including
  - pxelinux_no_append_block.t removed (superseded by write_pxelinux_config.t)
  - Several pxelinux_ks_xxx tests merged for easier maintenance (coverage
    and regexps tests are identical)
- Fixes quattor#258 (caused by a typo in the initial version of this commit)
@jouvin jouvin mentioned this issue Mar 22, 2017
Merged
@jrha
Copy link
Member

jrha commented Mar 27, 2017

@stdweird was this in master or were you testing #225?

@jrha jrha modified the milestones: 17.4, 17.3 Mar 27, 2017
@stdweird
Copy link
Member Author

testing #225, and is already fixed there. @jouvin wanted to keep this open, but it can already be closed afaik

@jouvin
Copy link
Contributor

jouvin commented Mar 27, 2017

#225 is supposed to close this issue... but I have no problem if you close it before...

jouvin added a commit to jouvin/aii that referenced this issue Mar 27, 2017
@jouvin
pxelinux: add kernel parameters in Grub2 config
4e91b3b 
- Includes some code refactoring/cleanup
- Unit tests adjusted to test PXELINUX and Grub2 variants, including
  - pxelinux_no_append_block.t removed (superseded by write_pxelinux_config.t)
  - Several pxelinux_ks_xxx tests merged for easier maintenance (coverage
    and regexps tests are identical)
- Fixes quattor#258 (caused by a typo in the initial version of this commit)
@jrha jrha closed this as completed in #225 Mar 29, 2017
@jrha jrha modified the milestones: 17.3, 17.4 Mar 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jouvin jouvin

Labels
None yet
Milestone
17.3
Development

No branches or pull requests
3 participants
@jrha @jouvin @stdweird