-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
linstor: use relative hostname path #8633
linstor: use relative hostname path #8633
Conversation
As described in issue apache#8310 some older distributions don't have hostname in /usr/bin so rely on PATH resolving
Thanks for fixing this @rp- |
@rohityadavcloud a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress. |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## 4.18 #8633 +/- ##
=========================================
Coverage 13.16% 13.16%
Complexity 9199 9199
=========================================
Files 2724 2724
Lines 258109 258109
Branches 40228 40228
=========================================
Hits 33970 33970
Misses 219837 219837
Partials 4302 4302 ☔ View full report in Codecov by Sentry. |
Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 8608 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
code lgtm
@@ -88,7 +88,7 @@ private String getHostname() { | |||
return result.trim(); | |||
} catch (IOException | InterruptedException exc) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
} catch (IOException | InterruptedException exc) { | |
} catch (Exception exc) { |
if so, remove IOException from import.
tested ok, merging |
As described in issue apache#8310 some older distributions don't have hostname in /usr/bin so rely on PATH resolving
As described in issue apache#8310 some older distributions don't have hostname in /usr/bin so rely on PATH resolving
As described in issue #8310 some older distributions don't have hostname in /usr/bin so rely on PATH resolving
Description
This PR Fixes: #8310 on 4.18 branch
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?
local linstor cluster and was working before see changes in #6682