8000 Fix usage of interfacename none if no interfacename is given by SchoolGuy · Pull Request #2683 · cobbler/cobbler · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix usage of interfacename none if no interfacename is given #2683

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

Merged
merged 1 commit into from
Jun 27, 2021

Conversation

SchoolGuy
Copy link
Member

Fixes #2681

@heroin-moose could you please confirm that this fixes the problem you discovered?

@SchoolGuy SchoolGuy added Bug Report Reporting a bug main Not a release but referring to the Git main branch labels Jun 23, 2021
@SchoolGuy SchoolGuy requested a review from nodeg June 23, 2021 07:18
@SchoolGuy SchoolGuy added this to the v3.3.0 milestone Jun 23, 2021
@heroin-moose
Copy link
Contributor

On it.

@heroin-moose
Copy link
Contributor

Yes, works for me.

@codecov
Copy link
codecov bot commented Jun 23, 2021

Codecov Report

Merging #2683 (4b9e63b) into master (4892d25) will not change coverage.
The diff coverage is 0.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2683   +/-   ##
=======================================
  Coverage   39.22%   39.22%           
=======================================
  Files          86       86           
  Lines       13363    13363           
=======================================
  Hits         5242     5242           
  Misses       8121     8121           
Impacted Files Coverage Δ
cobbler/items/system.py 74.26% <ø> (ø)
cobbler/remote.py 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4892d25...4b9e63b. Read the comment docs.

@SchoolGuy
Copy link
Member Author

Now waiting for the quality gatekeeper of our code @nodeg :)

@SchoolGuy
Copy link
Member Author

The reporter confirmed that this fix is working for him and requested that this may be merged. Thus not waiting for the review from @nodeg .

@SchoolGuy SchoolGuy merged commit bb089ad into master Jun 27, 2021
@SchoolGuy SchoolGuy deleted the fix/gh-2681 branch June 27, 2021 06:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Report Reporting a bug main Not a release but referring to the Git main branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New systems have two identical interfaces
3 participants
0