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.

Sign up for GitHub

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

Jump to bottom

TeamLocalityTest.GroupUnits fails if multiple units mapped to same domain #153

Open
fuchsto opened this issue Nov 23, 2016 · 2 comments
Open

TeamLocalityTest.GroupUnits fails if multiple units mapped to same domain #153

fuchsto opened this issue Nov 23, 2016 · 2 comments
Assignees
@fuchsto
Labels
enhancement feature known-limitation:0.3.0 module:team
Milestone
dash-0.4.0

Comments

@fuchsto
Copy link
Member

fuchsto commented Nov 23, 2016

No description provided.

@fuchsto fuchsto added the bug label Nov 23, 2016
@fuchsto fuchsto added this to the dash-0.3.0 milestone Nov 23, 2016
@fuchsto fuchsto self-assigned this Nov 23, 2016
@devreal
Copy link
Member

devreal commented Nov 23, 2016

Log Files:
team_locality_build.txt
team_locality_run.txt

@fuchsto
Copy link
Member Author

fuchsto commented Nov 23, 2016

Thanks, that confirms my assumption: multiple units are mapped to the same locality domain.

See team_locality_run.txt, line 5489:

[ ... TeamLocalityTest.cc :253 | TeamLocalityTest.SplitGroups | group 1: { .0.0.0.0.0.0.0, .0.0.0.0.1.0.0 } 
[ ... TeamLocalityTest.cc :254 | TeamLocalityTest.SplitGroups | group 2: { .0.0.0.0.1.0.0 }

This corresponds to the locality hierarchy at line 5383 ff.

@fuchsto fuchsto mentioned this issue Nov 23, 2016
Merged
@fuchsto fuchsto mentioned this issue Nov 28, 2016
Closed
@fuchsto fuchsto modified the milestones: dash-0.4.0, dash-0.3.0 Mar 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fuchsto fuchsto

Labels
enhancement feature known-limitation:0.3.0 module:team
Projects
None yet
Milestone
dash-0.4.0
Development

No branches or pull requests
3 participants
@fuchsto @fmoessbauer @devreal