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

GsDevKit/GsDevKit#60 patch should be included in GsUpgrader #14

Closed
dalehenrich opened this issue Mar 23, 2015 · 1 comment
Closed

GsDevKit/GsDevKit#60 patch should be included in GsUpgrader #14

dalehenrich opened this issue Mar 23, 2015 · 1 comment

Comments

@dalehenrich
Copy link
Member

My load testing with the GsDevKit sampleProject still exhibits occasional hangs prior to the official fix being loaded.

Since GsDevKit/GsDevKit#60 does not get addressed until the fix is loaded pretty late in the game, it will be useful to patch this bug at strategic points during the upgrade process ...
@dalehenrich dalehenrich mentioned this issue Mar 24, 2015
Closed
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: first cut at patch for GsDevKit/GsDevKit#60 in GsUpgrader ...
f91fa93
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: second cut at patch for GsDevKit/GsDevKit#60 in GsUpgrader…
408c28c 
… ... in fact it apears that a single 500 ms wait _without retries_ is not sufficient, so bugfix needs to change as well
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: debugging ????
94fbde0
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: binary search for timeout value...
79af381
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: ... searching
c11b0b7
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: settle on 20 seconds timeout
d3dc6a4
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: on further review the code was not retrying ... one chance…
630376f 
… only and we're hung and fail after timeout
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: okay now retry should work
cd4e00a
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: now that we are retrying 20 seconds per try is too long fo…
7d2e87d 
…r a connection that is HUNG by comcast
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: binary search
5a284e3
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: binary search
57ca7b1
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: spelunking ...
1067916
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: alternate route to getting information about the failure
702a885
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: shorter timeout for grins
27c51c0
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: timeout too short ...
35319e7
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issu #14: put back some (hopeful) error logging
4d9b56d
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: change of tack ... use a 20 second timeout and add retry l…
f567f28 
…ogic to MCHttpRepository>>allFileNames ... Gofer>>load appears to be the vulnerable party here ...
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: change of tack again ... MCPlatformSupport class>>readStre…
4d27818 
…amForUrl:user:passwd:do: is a better choice for retry logic as it covers Gofer allFileNames and package load ...
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: add mcz file
8ec554a
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: fix a bug
8aea1a6
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14 ... fix another flavor of same bug
24b9637
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: by jove I think we've got it ...
f0f898f
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: ... almost there
d8dfbbf
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: I can almost taste it ...
092e028
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: okay it looks like timeout needs to be bumped up ... when …
0612962 
…it rains it pours
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: zeroing in on the problem or circling the drain?
ab7a98e
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: crank down the timeout back to 20 s
2976962
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: loggings and messina
60bfb5b
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: now we are cooking with gas
e092b0d
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: cycling and circling
40e865d
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: By jove I think I've got ... for real ... no kidding
a035669
dalehenrich added a commit that referenced this issue Mar 24, 2015
@dalehenrich
Issue #14: yes this mcz file does seem to contain a right a and prope…
9a99fe9 
…r patch
@dalehenrich
Copy link
Member Author

green build

@dalehenrich dalehenrich mentioned this issue Mar 24, 2015
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@dalehenrich