Skip to content

Error if two hosts have the same hostname#1630

Merged
stevenengler merged 2 commits intoshadow:mainfrom
stevenengler:error-on-dup-hosts
Sep 7, 2021
Merged

Error if two hosts have the same hostname#1630
stevenengler merged 2 commits intoshadow:mainfrom
stevenengler:error-on-dup-hosts

Conversation

@stevenengler
Copy link
Copy Markdown
Contributor

@stevenengler stevenengler commented Sep 7, 2021

The previous behaviour was that one host would be "overwritten" by the other with the same name.

@stevenengler stevenengler added the Type: Bug Error or flaw producing unexpected results label Sep 7, 2021
@stevenengler stevenengler self-assigned this Sep 7, 2021
@github-actions github-actions bot added Component: Build Build/install tools and dependencies Component: Main Composing the core Shadow executable Component: Testing Unit and integration tests and frameworks labels Sep 7, 2021
@codecov
Copy link
Copy Markdown

codecov bot commented Sep 7, 2021

Codecov Report

Merging #1630 (fe85a1f) into main (2dbbedd) will increase coverage by 0.09%.
The diff coverage is 90.90%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1630      +/-   ##
==========================================
+ Coverage   54.44%   54.53%   +0.09%     
==========================================
  Files         141      141              
  Lines       18610    18616       +6     
  Branches     4466     4468       +2     
==========================================
+ Hits        10132    10153      +21     
+ Misses       5798     5789       -9     
+ Partials     2680     2674       -6     
Flag Coverage Δ
tests 54.53% <90.90%> (+0.09%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/main/core/controller.c 72.50% <75.00%> (+4.63%) ⬆️
src/main/core/manager.c 70.10% <100.00%> (ø)
src/main/core/scheduler/scheduler.c 74.34% <100.00%> (+0.54%) ⬆️
src/main/host/syscall/mman.c 62.12% <0.00%> (-1.52%) ⬇️
src/main/host/tracker.c 38.64% <0.00%> (-0.34%) ⬇️
src/main/host/syscall/socket.c 78.63% <0.00%> (-0.15%) ⬇️
src/main/host/descriptor/epoll.c 77.45% <0.00%> (ø)
src/main/host/descriptor/file.c 29.47% <0.00%> (+0.18%) ⬆️
src/main/host/network_interface.c 72.75% <0.00%> (+0.28%) ⬆️
src/main/host/descriptor/descriptor.c 75.00% <0.00%> (+0.73%) ⬆️
... and 3 more

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 2dbbedd...fe85a1f. Read the comment docs.

@stevenengler stevenengler merged commit 1678d00 into shadow:main Sep 7, 2021
@stevenengler stevenengler deleted the error-on-dup-hosts branch September 7, 2021 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component: Build Build/install tools and dependencies Component: Main Composing the core Shadow executable Component: Testing Unit and integration tests and frameworks Type: Bug Error or flaw producing unexpected results

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants