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

MTL-2380 Fix carriage returns #89

Merged
merged 2 commits into from
Feb 28, 2024
Merged

MTL-2380 Fix carriage returns #89

merged 2 commits into from
Feb 28, 2024

Conversation

rustydb
Copy link
Contributor

@rustydb rustydb commented Feb 27, 2024

Fix carriage returns to properly handle newlines when a single bracket is used in the YAML.

@rustydb rustydb requested a review from a team as a code owner February 27, 2024 20:13
@rustydb rustydb force-pushed the MTL-2380 branch 2 times, most recently from 75277ae to 7a95f6e Compare February 27, 2024 20:14
Fix carriage returns to properly handle newlines when a single bracket
is used in the YAML.
Handle cases where there is no network interface with the VIP index.
By default the VIP index is `2`, meaning systems without a `net2` will
end up failing this for-loop.

Regardless of what the VIP index is, ensure we fallback to the
iPXE's natural starting index if the VIP does not exist.
@rustydb rustydb requested a review from mtupitsyn February 27, 2024 21:23
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@mharding-hpe mharding-hpe changed the title MTL-2380 Fix carridge returns MTL-2380 Fix carriage returns Feb 28, 2024
@rustydb rustydb merged commit 467a257 into develop Feb 28, 2024
7 checks passed
@rustydb rustydb deleted the MTL-2380 branch February 28, 2024 19:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants