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

Ballot pages for some documents that have passed IESG are funny #1587

Closed
ietf-svn-bot opened this issue Jan 26, 2015 · 7 comments
Closed

Ballot pages for some documents that have passed IESG are funny #1587

ietf-svn-bot opened this issue Jan 26, 2015 · 7 comments

Comments

@ietf-svn-bot
Copy link

resolution_fixed type_defect | by paul.hoffman@vpnc.org


At this moment, the page for https://datatracker.ietf.org/doc/draft-ietf-dnsop-child-syncronization/ballot/ says:

Note: This ballot was opened for revision 02 and is now closed.

Summary: Needs 100 more YES or NO OBJECTION positions to pass.


Issue migrated from trac:1587 at 2022-03-04 03:59:45 +0000

@ietf-svn-bot
Copy link
Author

@rjsparks@nostrum.com commented


Yes, the summary line should not present for documents in this state

@ietf-svn-bot
Copy link
Author

@rjsparks@nostrum.com commented


This isn't just documents that have closed ballots.
Right now https://datatracker.ietf.org/doc/draft-ietf-savi-dhcp/ballot/ claims it needs 111 positions.

This is due to the changes in the recent release for supporting out-of-area ADs.
I'll work on a patch.

@ietf-svn-bot
Copy link
Author

@rjsparks@nostrum.com changed status from new to closed

@ietf-svn-bot
Copy link
Author

@rjsparks@nostrum.com changed resolution from `` to fixed

@ietf-svn-bot
Copy link
Author

@rjsparks@nostrum.com commented


Fixed in 7e188d2:

Fix needed-to-pas calculation to use new representation of area directors. Fixes #1587. Commit ready to merge

@ietf-svn-bot
Copy link
Author

@rjsparks@nostrum.com commented


Patch applied to production

@ietf-svn-bot
Copy link
Author

@henrik@levkowetz.com commented


Fixed in 278b30c:

Merged in 7e188d2 from rjsparks@nostrum.com:
Fix needed-to-pas calculation to use new representation of area directors. Fixes #1587.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant