{"action":"created","issue":{"url":"https://api.github.com/repos/OpenLiberty/open-liberty/issues/6025","repository_url":"https://api.github.com/repos/OpenLiberty/open-liberty","labels_url":"https://api.github.com/repos/OpenLiberty/open-liberty/issues/6025/labels{/name}","comments_url":"https://api.github.com/repos/OpenLiberty/open-liberty/issues/6025/comments","events_url":"https://api.github.com/repos/OpenLiberty/open-liberty/issues/6025/events","html_url":"https://github.com/OpenLiberty/open-liberty/issues/6025","id":390801918,"node_id":"MDU6SXNzdWUzOTA4MDE5MTg=","number":6025,"title":"Comment out oddly failing unit tests while further debugging","user":{"login":"billw1024","id":29730989,"node_id":"MDQ6VXNlcjI5NzMwOTg5","avatar_url":"https://avatars1.githubusercontent.com/u/29730989?v=4","gravatar_id":"","url":"https://api.github.com/users/billw1024","html_url":"https://github.com/billw1024","followers_url":"https://api.github.com/users/billw1024/followers","following_url":"https://api.github.com/users/billw1024/following{/other_user}","gists_url":"https://api.github.com/users/billw1024/gists{/gist_id}","starred_url":"https://api.github.com/users/billw1024/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/billw1024/subscriptions","organizations_url":"https://api.github.com/users/billw1024/orgs","repos_url":"https://api.github.com/users/billw1024/repos","events_url":"https://api.github.com/users/billw1024/events{/privacy}","received_events_url":"https://api.github.com/users/billw1024/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2018-12-13T18:21:57Z","updated_at":"2018-12-14T16:57:18Z","closed_at":null,"author_association":"MEMBER","body":"Some vhost unit tests are failing that have worked for years, and no changes to the tests or vhost code has been made recently.  So for now will comment out these test, and then try to figure out what is the issue.\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/OpenLiberty/open-liberty/issues/comments/447385836","html_url":"https://github.com/OpenLiberty/open-liberty/issues/6025#issuecomment-447385836","issue_url":"https://api.github.com/repos/OpenLiberty/open-liberty/issues/6025","id":447385836,"node_id":"MDEyOklzc3VlQ29tbWVudDQ0NzM4NTgzNg==","user":{"login":"wtlucy","id":5764788,"node_id":"MDQ6VXNlcjU3NjQ3ODg=","avatar_url":"https://avatars2.githubusercontent.com/u/5764788?v=4","gravatar_id":"","url":"https://api.github.com/users/wtlucy","html_url":"https://github.com/wtlucy","followers_url":"https://api.github.com/users/wtlucy/followers","following_url":"https://api.github.com/users/wtlucy/following{/other_user}","gists_url":"https://api.github.com/users/wtlucy/gists{/gist_id}","starred_url":"https://api.github.com/users/wtlucy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wtlucy/subscriptions","organizations_url":"https://api.github.com/users/wtlucy/orgs","repos_url":"https://api.github.com/users/wtlucy/repos","events_url":"https://api.github.com/users/wtlucy/events{/privacy}","received_events_url":"https://api.github.com/users/wtlucy/received_events","type":"User","site_admin":false},"created_at":"2018-12-14T16:57:18Z","updated_at":"2018-12-14T16:57:18Z","author_association":"MEMBER","body":"The changes I made in PR #3981 affected this test scenario: we now log a message and alert listeners when a context root is added even if there are zero ports associated with the context root.  The failing test - VirtualHostMapTest.testThreadedAddRemoveMultipleVHosts  - assumes that two of the context roots it's working with (/z and /y) will only notify contextRootAdded listeners once.  When this test fails, it's because that a listener for one of those roots was notified more than once.  Before PR #3981 that would never happen, since the /z and /y might have had zero associated ports at the time of addContextRoot() was called; we should now tolerate multiple notifications for those roots."}}
{"actor":{"display_login":"wtlucy"}}