Fixup Integ Test + Limit Messages in Server Memory #36

병합
dan integ_test_fix 에서 master 로 1 commits 를 머지했습니다 2018-06-09 21:21:43 +00:00
소유자
No description provided.
소유자

? ms.rotated - Maybe "rolledOver"? or wrappedAround?

integ tests:

117 +		time.Sleep(time.Second * 120)

Not needed, the only thing under it is the starting of the 3 peers, and a currently 70 second for their onions to come online too. I stream lined the code a bit this way so they and the test server could come online all at once, before any peering ations were done. If you need just increase the sleep 70 at ~124 to 120

Why did you move the server join down? I had them fire off all those actions in a group so it was streamlined so the integ test ran a bit faster :)

(your 183...)
also the tests shouldnt use log (did t.log actually work tho? maybe I just muted log.) because the first thing i do at the top of the test is mute log since I don't want to see all the trace log code

We may want to make waitForPeerServerConnection and waitForPeerPeerConnection then and remove even more sleeps

? ms.rotated - Maybe "rolledOver"? or wrappedAround? integ tests: 117 + time.Sleep(time.Second * 120) Not needed, the only thing under it is the starting of the 3 peers, and a currently 70 second for their onions to come online too. I stream lined the code a bit this way so they and the test server could come online all at once, before any peering ations were done. If you need just increase the sleep 70 at ~124 to 120 Why did you move the server join down? I had them fire off all those actions in a group so it was streamlined so the integ test ran a bit faster :) (your 183...) also the tests shouldnt use log (did t.log actually work tho? maybe I just muted log.) because the first thing i do at the top of the test is mute log since I don't want to see all the trace log code We may want to make waitForPeerServerConnection and waitForPeerPeerConnection then and remove even more sleeps
소유자

Also I think the file storage needs reworking, we are still just filling the file infinitely and on reload, its still appending to the buffer so it will increase its size well beyond max size

Also I think the file storage needs reworking, we are still just filling the file infinitely and on reload, its still appending to the buffer so it will increase its size well beyond max size
Author
소유자

Yeah I mentioned in the previous PR that I will fixup the file stuff later "This adds a basic circular buffer for messages, still need to rotate the underlying file, but this will prevent memory getting too bloated (partial fix for #22)" handling the file stuff is going to involve much more rework.

t.Log works, the muted log is just for global log stuff in the rest of the library.

I think the Sleeps might be a a debugging thing, I'll remove and retest.

Yeah I mentioned in the previous PR that I will fixup the file stuff later "This adds a basic circular buffer for messages, still need to rotate the underlying file, but this will prevent memory getting too bloated (partial fix for #22)" handling the file stuff is going to involve much more rework. t.Log works, the muted log is just for global log stuff in the rest of the library. I think the Sleeps might be a a debugging thing, I'll remove and retest.
소유자

Then maybe at least comment out the init code loading from the file cus I think it totally breaks everything :)

Then maybe at least comment out the init code loading from the file cus I think it totally breaks everything :)
Author
소유자

Removed sleeps and put calls back to where they were, reverted t.Log.

We talked about the server stuff offline, leaving unchanged.

Removed sleeps and put calls back to where they were, reverted t.Log. We talked about the server stuff offline, leaving unchanged.
dan closed this pull request 2018-06-09 21:21:43 +00:00
"로그인하여 이 대화에 참여"
No reviewers
마일스톤 없음
담당자 없음
참여자 2명
알림
마감일
기한이 올바르지 않거나 범위를 벗어났습니다. 'yyyy-mm-dd'형식을 사용해주십시오.

마감일이 설정되지 않았습니다.

의존성

No dependencies set.

Reference: cwtch.im/cwtch#36
No description provided.