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.

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

host_error hook must take arguments from standard input instead of command line #5101

Closed
7 tasks
OpenNebulaSupport opened this issue Sep 21, 2020 · 0 comments
Closed
7 tasks

Comments

@OpenNebulaSupport
Copy link
Collaborator

Description
The host_error.rb script needs to receive the host template to work. Currently the host template is passed as argument from the command line. This could cause an potential overflow problem because of the host template length.

To Reproduce
Run the host_error.rb script with a large host template string as command line argument.

Expected behavior
The script host_error.rb must receive the host template from standard input to avoid the command line argument length limitation.

Details

  • Affected Component: Hooks
  • Hypervisor: KVM
  • Version: 5.12

Progress Status

  • Branch created
  • Code committed to development branch
  • Testing - QA
  • Documentation
  • Release notes - resolved issues, compatibility, known issues
  • Code committed to upstream release/hotfix branches
  • Documentation committed to upstream release/hotfix branches
@OpenNebulaSupport OpenNebulaSupport added this to the Release 5.12.4 milestone Sep 21, 2020
@tinova tinova modified the milestones: Release 5.12.4, Release 5.12.5 Sep 23, 2020
rsmontero added a commit that referenced this issue Sep 24, 2020
Co-authored-by: Ruben S. Montero <[email protected]>
Signed-off-by: Ricardo Diaz <[email protected]>
atodorov-storpool pushed a commit to storpool/one that referenced this issue Oct 15, 2020
atodorov-storpool pushed a commit to storpool/one that referenced this issue Oct 16, 2020
atodorov-storpool pushed a commit to storpool/one that referenced this issue Oct 16, 2020
@tinova tinova modified the milestones: Release 5.12.5, Release 5.12.6 Oct 21, 2020
@tinova tinova modified the milestones: Release 5.12.6, Release 5.12.7 Nov 5, 2020
@tinova tinova modified the milestones: Release 5.12.7, Release 6.0 Nov 18, 2020
atodorov-storpool pushed a commit to storpool/one that referenced this issue Nov 27, 2020
atodorov-storpool pushed a commit to storpool/one that referenced this issue Nov 27, 2020
atodorov-storpool pushed a commit to storpool/one that referenced this issue Jan 4, 2021
atodorov-storpool pushed a commit to storpool/one that referenced this issue Jan 7, 2021
rsmontero added a commit that referenced this issue Jan 20, 2021
Co-authored-by: Ruben S. Montero <[email protected]>
Signed-off-by: Ricardo Diaz <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants