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

Server count rollups should handle duplicate host names #409

Closed
rovats opened this issue Aug 8, 2016 · 1 comment
Closed

Server count rollups should handle duplicate host names #409

rovats opened this issue Aug 8, 2016 · 1 comment
Labels
fixed (test & close) An issue has been fixed, merged into master and is ready for further testing
Milestone
0.6.0

Comments

@rovats
Copy link
Member

rovats commented Aug 8, 2016 •
edited
Loading

Some servers can be associated with multiple applications, so the rollups need to take distinct server counts.

  • Also change server information sample generator to include duplicate server names (associated with different apps)
@rovats
Copy link
Member Author

rovats commented Sep 13, 2016

wip

rovats added a commit to rovats/waltz that referenced this issue Sep 13, 2016
@rovats
Server count rollups should handle duplicate host names
1ccc62c 
finos#409
@rovats rovats mentioned this issue Sep 13, 2016
Merged
rovats added a commit to rovats/waltz that referenced this issue Sep 13, 2016
@rovats
Rename variable for clarity
e8229e8 
finos#409
@davidwatkins73 davidwatkins73 added the fixed (test & close) An issue has been fixed, merged into master and is ready for further testing label Sep 13, 2016
rovats added a commit to rovats/waltz that referenced this issue Sep 14, 2016
@rovats
Fix bug when running under SQL Server
733471a 
finos#409
@rovats rovats mentioned this issue Sep 14, 2016
Merged
@davidwatkins73 davidwatkins73 modified the milestone: 0.6.0 Sep 16, 2016
plexus-user pushed a commit to deutschebank/waltz that referenced this issue May 28, 2020
@MarkGuerriero
Merge pull request finos#409 in WALTZ/waltz-gh-staging from WALTZ/wal…
e2a9cc7 
…tz-ma:CTCTOWALTZ-1761_update_rating_logs to waltz-4875_update_audit_logs_ratings

* commit '67e10d6209c76ca129c55685f57a49eb266605da':
  changed externalId property as readonly
  Added audit log for rating
deutschebank-sync pushed a commit to deutschebank/waltz that referenced this issue Oct 21, 2024
@jain-shreyans-db
Pull request finos#409: remove unused imports, and change test assert…
42afe49 
…ion bug

Merge in WALTZ/waltz from WALTZ/waltz-sj:CTCTOWALTZ-3415-variable-reference-issue-3 to db-feature/entity-relationship-variable-ref-fix-7175

* commit 'a1c6fa6c3093c78d5227e5a6c90994068fc224c0':
  remove unused imports, and change test assertion bug
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
fixed (test & close) An issue has been fixed, merged into master and is ready for further testing
Projects
None yet
Milestone
0.6.0
Development

No branches or pull requests
2 participants
@davidwatkins73 @rovats