[11:01:07 CDT(-0500)] <JasonElwood> signing off for a few hours to work on IRSC configuration
[11:01:22 CDT(-0500)] <dmccallum54> JasonElwood… SSP-767
[11:01:27 CDT(-0500)] <dmccallum54> before i lose you...
[11:01:42 CDT(-0500)] <dmccallum54> your email said that was checked into master but needs to be in rel-1-23
[11:01:46 CDT(-0500)] <dmccallum54> rel-1-2
[11:01:56 CDT(-0500)] <dmccallum54> but looks to me like it's a giant ticket for Main tool 2.0
[11:03:25 CDT(-0500)] <JasonElwood> yeah. I incorrectly identified the issues.
[11:03:56 CDT(-0500)] <JasonElwood> one sec
[11:06:43 CDT(-0500)] <JasonElwood> It should be 677, 769 and 757. 767 and 787 are ok
[11:07:20 CDT(-0500)] <dmccallum54> ok
[11:07:21 CDT(-0500)] <dmccallum54> then ignore that question in my email
[11:07:43 CDT(-0500)] <dmccallum54> will look at 769
[11:08:01 CDT(-0500)] <dmccallum54> gah. nm. already commented on that one...
[11:08:30 CDT(-0500)] <JasonElwood> 769 could be moved to master OR add the fields to MyGPS
[11:08:52 CDT(-0500)] <dmccallum54> 787 is OK or that needs to be moved to master?
[11:08:53 CDT(-0500)] <dmccallum54> fix version is 2.0.x?
[11:09:14 CDT(-0500)] <dmccallum54> TonyUnicon might feel differently, but my sense was that we shouldnt be signing up for new GPS features for 1.2.0
[11:09:25 CDT(-0500)] <TonyUnicon> yes
[11:09:31 CDT(-0500)] <JasonElwood> I would agree
[11:09:31 CDT(-0500)] <TonyUnicon> I agree
[11:09:40 CDT(-0500)] <JasonElwood> if that is the case 787 should be in master
[11:09:51 CDT(-0500)] <JasonElwood> it is defined for 2.0.x which is fine
[11:09:56 CDT(-0500)] <dmccallum54> ok. will move 769 and 787 to master and out of rel-1-2 then
[11:10:18 CDT(-0500)] <dmccallum54> which then leaves just 677 and 757 to be ported from master to 1-2
[11:10:35 CDT(-0500)] <dmccallum54> and 767 i'll just ignore
[11:10:44 CDT(-0500)] <JasonElwood> yep
[11:10:55 CDT(-0500)] <dmccallum54> k. thx
[11:11:11 CDT(-0500)] <JasonElwood> sorry. way to many balls in the air right now
[11:12:35 CDT(-0500)] <dmccallum54> welcome to the club
[11:14:23 CDT(-0500)] <JasonElwood> I prefer that you guys join my wolf pack
[11:15:07 CDT(-0500)] <js70> Apologize for putting 787 in 1.2 saw the affects version1.2.x and current iteration and went from there.
[11:15:21 CDT(-0500)] <js70> need to look at the fixed version
[11:21:59 CDT(-0500)] <TonyUnicon> Jason no matter how hard I try I cannot reproduce SSP-826
[11:22:08 CDT(-0500)] <TonyUnicon> even following your new steps
[11:22:40 CDT(-0500)] <TonyUnicon> Do you think when you have a chance you can get the same user back in that state and track exactly what you did (down to the field)
[11:22:53 CDT(-0500)] <TonyUnicon> and ive tried on both envs
[11:28:26 CDT(-0500)] <JasonElwood> it's going to be a while
[12:11:24 CDT(-0500)] <js701> http://localhost:8080/ssp/api/1/person/477497f2-cbcb-4269-91dc-50d0e9ba224b/journalEntry/3e3edcf7-ac75-478f-bddf-b5743327d12aFailed to load resource: the server responded with a status of 404 (Not Found)
[12:11:44 CDT(-0500)] <js701> got this when trying to update a journal entry. any ideas?
[12:13:05 CDT(-0500)] <dmccallum54> dunno
[12:13:13 CDT(-0500)] <dmccallum54> is it actually hitting the controller method
[12:14:27 CDT(-0500)] <js701> just a sec. brains not working. debugger is not on.
[13:13:55 CDT(-0500)] <dmccallum54> TonyUnicon 437b87e2b0144f42ba7ea747ddc602eb4c12c123 adds references to 6 liquibase changesets. 3 of them dont exist yet in master. one of them (000022.xml) is being dealt with as part of the Great 1.2 Patch Relocation Effort. what's up with the other two (i000007.xml and i000008.xml)?
[13:27:58 CDT(-0500)] <js701> i000007.xml and i000008.xml are mine
[13:28:29 CDT(-0500)] <js701> non_local_address for i00007.xml and photo_url for i00008.xml
[13:28:52 CDT(-0500)] <dmccallum54> they're not in source
[13:29:08 CDT(-0500)] <dmccallum54> or not in master anyway
[13:29:09 CDT(-0500)] <js701> hmmm. let me look
[13:29:10 CDT(-0500)] <dmccallum54> ok
[13:29:19 CDT(-0500)] <js701> I think they are in 1.2
[13:29:46 CDT(-0500)] <dmccallum54> ok. i see them there
[13:30:00 CDT(-0500)] <dmccallum54> alright
[13:30:01 CDT(-0500)] <dmccallum54> well
[13:30:30 CDT(-0500)] <dmccallum54> i think i'm going to delete the bad references in master
[13:30:43 CDT(-0500)] <dmccallum54> but then we might have a bit of a problem in people's local builds
[13:31:00 CDT(-0500)] <dmccallum54> b/c tony's commit adds a reference to 000023.xml which does exist in master
[13:31:16 CDT(-0500)] <dmccallum54> but now that's going to run before 000022.xml
[13:31:50 CDT(-0500)] <dmccallum54> well…. actually…. if you have a separate db for master, you should be fine
[13:32:14 CDT(-0500)] <dmccallum54> and i think i'll just skip 22 and go right for 24
[13:32:38 CDT(-0500)] <dmccallum54> still think we might have a problem in people's local 1.2 builds but we'll see
[14:11:27 CDT(-0500)] <dmccallum54> ok SSP-769, SSP-787 have been pushed to master. need to step away for a moment but then will start on undoing them in rel-1-2
[14:28:59 CDT(-0500)] <TonyUnicon> ahh did I accidentally push a 1.2 change to master?
[14:29:09 CDT(-0500)] <dmccallum54> seems so
[14:29:21 CDT(-0500)] <dmccallum54> think its all fixed up now
[14:29:30 CDT(-0500)] <dmccallum54> the liquibase portion anyway
[14:30:50 CDT(-0500)] <TonyUnicon> :/
[14:35:00 CDT(-0500)] <TonyUnicon> there is another portion?
[14:35:10 CDT(-0500)] <dmccallum54> dont know
[14:39:13 CDT(-0500)] <dmccallum54> the commit i was dealing with was 437b87e2b0144f42ba7ea747ddc602eb4c12c123, which definitely had 1-2-related lines.
[14:39:32 CDT(-0500)] <dmccallum54> whether there were other commits that also brought in 1-2 changes inadvertently, i just dont know
[15:23:16 CDT(-0500)] <dmccallum541> pushed a revert of SSP-769, SSP-787 to rel-1-2 patches. it looks like my fears of liquibase hell were unfounded
[15:36:24 CDT(-0500)] <js701> good news!
[15:37:52 CDT(-0500)] <JasonElwood> Dan- for SSP-824 (email service), I'm good with your response. I was concerned that it did attempt to send the email but failed due to the malformed email address. My question was: why would it check everything to be sent and not send?
[15:39:09 CDT(-0500)] <dmccallum541> JasonElwood it's so it doesn't have to sweep an ever increasing number of unsent messages
[15:39:17 CDT(-0500)] <dmccallum541> also
[15:39:25 CDT(-0500)] <dmccallum541> let's say you turned off mail in a real deployment
[15:39:25 CDT(-0500)] <JasonElwood> cool. thanks
[15:39:32 CDT(-0500)] <dmccallum541> then after, i dunno, a week, you turn it back on
[15:39:42 CDT(-0500)] <dmccallum541> you probably don't want all the msgs that have queued up over that week to go out
[15:40:07 CDT(-0500)] <JasonElwood> closed it
[15:40:31 CDT(-0500)] <dmccallum541> it's a canned example i know, but it did actually work out nicely at CLC where they had email off when the system first came online
[15:40:36 CDT(-0500)] <dmccallum541> then turned it on several days later
[15:40:58 CDT(-0500)] <JasonElwood> IRSC wants to do something similar
[15:41:12 CDT(-0500)] <dmccallum541> sweet
[15:41:21 CDT(-0500)] <dmccallum541> how'd the training/walkthrough go?
[15:41:30 CDT(-0500)] <JasonElwood> I think like SPC where all emails went to a specific address to test, then turn on in prod
[15:42:04 CDT(-0500)] <JasonElwood> It went fine. There were 5-6 people involved which is good because they are thinking about implementation decisions now and not during training
[15:42:14 CDT(-0500)] <JasonElwood> they want a lot more data loaded from SIS
[15:47:42 CDT(-0500)] <dmccallum541> for email… SPC had mail turned on, but they set up a rule in Exchange to route everything from the SSP dev 'from' address to Patrick Booth
[15:48:11 CDT(-0500)] <dmccallum541> CLC was less ambitious… they just temporarily modified email addresses in SSP
[15:50:05 CDT(-0500)] <js701> Jason for the Disability Services Report, is there any chance the person you were looking up or expected to see either did not have a Student Type was not assoicated to a coach or did not have a disability? Those are the only criteria that I apply.
[15:51:43 CDT(-0500)] <JasonElwood> Jim- I'll check but I doubt it. I just added a disability to an existing student who showed up in other reports
[15:52:04 CDT(-0500)] <JasonElwood> Dan- we'll need to explore it further for IRSC
[15:52:18 CDT(-0500)] <js701> k. Its just I am see students show up on my reports.
[15:54:10 CDT(-0500)] <JasonElwood> Jim- I checked. All conditions apply
[15:59:16 CDT(-0500)] <JasonElwood> Jim- I just emailed the disability report and some screen shots
[16:01:56 CDT(-0500)] <js701> k
[16:04:56 CDT(-0500)] <dmccallum541> JasonElwood have you seen any of our famous "java.lang.RuntimeException: java.lang.IllegalStateException: Neither HttpServletResponse.getWriter nor HttpServletResponse.getOutputStream has not been called" recently? A-la SSP-172 or SSP-812?
[16:05:40 CDT(-0500)] <JasonElwood> i did yesterday in testing reports. I haven't tested reports yet today. Jim has some changes that I can check in a little bit.
[16:05:44 CDT(-0500)] <dmccallum541> (i like how even just parsing the error message's double negative adds to the challenge of tracking that one down)
[16:06:04 CDT(-0500)] <JasonElwood> trying to work through the MyGPS use case
[16:06:11 CDT(-0500)] <dmccallum541> no rush
[16:06:58 CDT(-0500)] <dmccallum541> whenever you get a chance… what's the likelihood you still have the command you used for your most recent platform deploy still in your history
[16:07:09 CDT(-0500)] <dmccallum541> i'm wondering if it was a deploy-ear or initportal without a clean
[16:08:24 CDT(-0500)] <JasonElwood> last platform was initportal. it was to catch some changes tony made
[16:12:00 CDT(-0500)] <dmccallum541> 'clean initportal' or just 'initportal'. if the latter, and you go grab a snack at somepoint and want to redeploy with the former, or even just 'clean deploy-ear' or 'clean deploy-war', i think it might be helpful
[16:12:29 CDT(-0500)] <dmccallum541> but i know you never leave your desk
[16:12:32 CDT(-0500)] <dmccallum541> so that's probably out...
[16:13:03 CDT(-0500)] <JasonElwood> Yeah. but I'll still take the snack. I'll wait to my wife gets home, and she can deliver a snack.
[16:13:38 CDT(-0500)] <dmccallum541> TonyUnicon i'm grabbing SSP-711 unless you've already started it
[16:13:46 CDT(-0500)] <JasonElwood> it would have been initportal
[16:14:14 CDT(-0500)] <TonyUnicon> i have not
[16:14:28 CDT(-0500)] <TonyUnicon> but if you're free
[16:14:33 CDT(-0500)] <TonyUnicon> there is something I can use a second eye on
[16:14:39 CDT(-0500)] <TonyUnicon> 711 is just a change in the date formatter
[16:14:42 CDT(-0500)] <TonyUnicon> I can handle that
[16:14:43 CDT(-0500)] <dmccallum541> JasonElwood the reason i'm suggesting a redeploy with 'clean' … i was getting that error all the time until i deployed the uPortal half of the 688 fix and i haven't seen it since.
[16:14:50 CDT(-0500)] <dmccallum541> and i always run a clean first
[16:14:59 CDT(-0500)] <dmccallum541> as does greg, b/c he's been bitten by not doing that in other cases
[16:15:19 CDT(-0500)] <dmccallum541> and greg's not been able to reproduce the issue
[16:15:40 CDT(-0500)] <JasonElwood> I'd like to try clean deploy first so I don't lose my uPortal users
[16:15:41 CDT(-0500)] <dmccallum541> Jim, who also sees the issue, also does not, i don't think, use clean when he deploys uportal
[16:15:53 CDT(-0500)] <dmccallum541> JasonElwood that's fine
[16:15:56 CDT(-0500)] <dmccallum541> i dont think it's a db issue
[16:16:00 CDT(-0500)] <dmccallum541> just a stale binary issue
[16:16:08 CDT(-0500)] <dmccallum541> so no need to blow away your uP db
[16:16:21 CDT(-0500)] <dmccallum541> TonyUnicon yeah 711 was easy-pickins
[16:16:28 CDT(-0500)] <dmccallum541> if there's something else you want me to grab, i will
[16:16:33 CDT(-0500)] <TonyUnicon> I could use some help on SSP-822
[16:16:37 CDT(-0500)] <JasonElwood> I do occasionally undeploy both ssp-platform and ssp in tomcat
[16:16:53 CDT(-0500)] <TonyUnicon> I have no clue what the issue is here
[16:17:05 CDT(-0500)] <TonyUnicon> the API call hits the server where it's suppose to
[16:17:17 CDT(-0500)] <TonyUnicon> the controller method executes without error
[16:17:21 CDT(-0500)] <TonyUnicon> but the response comes back 404
[16:17:26 CDT(-0500)] <TonyUnicon> no clue
[16:17:49 CDT(-0500)] <TonyUnicon> iI would update before taking a look
[16:18:02 CDT(-0500)] <TonyUnicon> it did throw a null pointer under certain circumstances and that has been addressed
[16:18:06 CDT(-0500)] <TonyUnicon> but it does not fix the greater issue
[16:18:07 CDT(-0500)] <dmccallum541> JasonElwood the (weak) theory is stale binaries in the build, not so much correct binaries in the build but incorrect binaries in tomcat. i.e. maven is for whatever reason skipping the change in 1307fb458d8d3e2648b8bc39a043e3e91a3e1654
[16:18:57 CDT(-0500)] <dmccallum541> ugh
[16:19:00 CDT(-0500)] <JasonElwood> got it. I did run an ant clean before the last initportal
[16:19:08 CDT(-0500)] <JasonElwood> no matter. I'll try again today
[16:19:40 CDT(-0500)] <dmccallum541> JasonElwood … let me try that again… the (weak) theory is stale binaries in the build, not so much incorrect binaries hanging around in tomcat for whatever reason, but incorrect binaries in the build. i.e. maven is for whatever reason skipping the change in 1307fb458d8d3e2648b8bc39a043e3e91a3e1654
[16:20:03 CDT(-0500)] <dmccallum541> TonyUnicon 822. got it
[16:28:22 CDT(-0500)] <dmccallum541> TonyUnicon… i have a theory...
[16:28:29 CDT(-0500)] <TonyUnicon> hmm?
[16:28:40 CDT(-0500)] <dmccallum541> missing @ResponseBody
[16:28:44 CDT(-0500)] <dmccallum541> on the controller mtd
[16:29:21 CDT(-0500)] <TonyUnicon> sonofa
[16:29:33 CDT(-0500)] <TonyUnicon> let me test
[16:29:41 CDT(-0500)] <JasonElwood> the ugly 500 error just struck in MyGPS
[16:30:18 CDT(-0500)] <TonyUnicon> do you remember what you did
[16:30:35 CDT(-0500)] <TonyUnicon> dmccallum541 testing your theory
[16:30:58 CDT(-0500)] <JasonElwood> I was completing a student intake in MyGPS, clicked save to complete
[16:31:07 CDT(-0500)] <JasonElwood> which I've done 10 times today without error
[16:31:35 CDT(-0500)] <TonyUnicon> i dont think you'd get the same error on save
[16:31:45 CDT(-0500)] <TonyUnicon> but thats awesome
[16:31:53 CDT(-0500)] <JasonElwood>
[16:32:20 CDT(-0500)] <TonyUnicon> yeah that is different
[16:32:42 CDT(-0500)] <TonyUnicon> maybe the session timed out?
[16:33:19 CDT(-0500)] <TonyUnicon> can you email me the entire stack
[16:34:13 CDT(-0500)] <JasonElwood> that's the same, just difference resource
[16:34:21 CDT(-0500)] <TonyUnicon> it is not the same
[16:34:28 CDT(-0500)] <dmccallum541> TonyUnicon the core issue for Jason's 500 is SSP-688
[16:34:29 CDT(-0500)] <TonyUnicon> the other 500 error was a null pointer
[16:34:53 CDT(-0500)] <dmccallum541> his current 500 i mean
[16:35:06 CDT(-0500)] <TonyUnicon> ok
[16:35:25 CDT(-0500)] <TonyUnicon> yeah its a timeout
[16:35:54 CDT(-0500)] <TonyUnicon> Jason this is not the same exception as https://issues.jasig.org/browse/SSP-826
[16:36:28 CDT(-0500)] <dmccallum541> TonyUnicon, it's not the @ResponseBody
[16:36:37 CDT(-0500)] <TonyUnicon> heh
[16:36:41 CDT(-0500)] <TonyUnicon> was just typing the same thing
[16:37:24 CDT(-0500)] <JasonElwood> two different MyGPS problems. the intake from this morning is not the one I just reported. The one I just reported is the one Dan wants the uportal clean to be done
[16:37:38 CDT(-0500)] <TonyUnicon> right
[16:38:07 CDT(-0500)] <TonyUnicon> but that is not a mygps problem that is a uportal problem
[16:38:22 CDT(-0500)] <JasonElwood> rigth
[16:43:30 CDT(-0500)] <dmccallum541> TonyUnicon my new theory is there isn't a view named "actionPlanReport"
[16:43:41 CDT(-0500)] <JasonElwood> Jim- your latest change on the disability report blows up the mvn build
[16:44:23 CDT(-0500)] <TonyUnicon> I think that is more likely
[16:44:36 CDT(-0500)] <dmccallum541> the @ResponseBody thing has definitely caused the same type of 404 behavior on "normal" API calls where we just pass back the object to be marshalled. but i see this method works differently (traditional ModelAndView return type)
[16:44:54 CDT(-0500)] <JasonElwood> [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:2.3.2:compile (default-compile) on project ssp: Compilation failure
[16:44:55 CDT(-0500)] <JasonElwood> [ERROR] \Apps\projects\SSP\src\main\java\org\jasig\ssp\dao\report\DisabilityServiceReportDao.java:[67,55] cannot find symbol
[16:44:55 CDT(-0500)] <JasonElwood> [ERROR] symbol : variable totalRows
[16:44:55 CDT(-0500)] <JasonElwood> [ERROR] location: class org.jasig.ssp.dao.report.DisabilityServiceReportDao
[16:45:15 CDT(-0500)] <dmccallum541> js701 that looks like you
[16:47:01 CDT(-0500)] <JasonElwood> sorry I copied over my first part. I just checked out the latest code. Jim- the change for the disability report didn't quite work
[16:49:06 CDT(-0500)] <js701> yeah i know. I am checking in the compile error. my apologies
[16:50:07 CDT(-0500)] <dmccallum541> TonyUnicon is /ssp/api/1/mygps/task/print supposed to output the same thing as /ssp/api/1/person/ /task/print, but with some extra magic around looking up a task list for an anonymous user?
[16:50:20 CDT(-0500)] <JasonElwood> if it happens again, you have to swap reports and MyGPS with Tony
[16:50:59 CDT(-0500)] <js701> NOoooooooo! I LOOOOOVE Reports! ^o)
[16:51:02 CDT(-0500)] <TonyUnicon> i have not touched anything involving printing
[16:51:07 CDT(-0500)] <js701> changes checked in
[16:51:25 CDT(-0500)] <TonyUnicon> i haven't looked at that other API call
[16:51:56 CDT(-0500)] <TonyUnicon> Hey now these MyGPS bugs were coded months ago, im just the sucker to inherited them
[16:52:35 CDT(-0500)] <JasonElwood> right. I was trying to punish Jim by giving him MyGPS
[16:52:42 CDT(-0500)] <JasonElwood> but you are welcome to keep it
[16:52:45 CDT(-0500)] <TonyUnicon> oh well thats well and good
[16:52:50 CDT(-0500)] <TonyUnicon> 
[16:56:51 CDT(-0500)] <TonyUnicon> i have to run in about 15 minutes but I'll be back in about 3 hours
[16:57:14 CDT(-0500)] <TonyUnicon> 711 is checked in
[16:57:27 CDT(-0500)] <dmccallum541> JasonElwood is the GPS task report supposed to be exactly the same thing you get if you click the print icon in the Action Plan tool in the Ext.js UI?
[16:58:16 CDT(-0500)] <JasonElwood> funny you should ask, I don't know. I've never seen it work
[16:58:23 CDT(-0500)] <dmccallum541> and…. i should probably know this one… do we actually support anonymous task lists in GPS?
[16:58:39 CDT(-0500)] <JasonElwood> we do support anonymous task lists
[16:58:42 CDT(-0500)] <dmccallum541> ok
[16:58:59 CDT(-0500)] <JasonElwood> I'm checking
[17:01:06 CDT(-0500)] <JasonElwood> wait. I got one to work.
[17:01:08 CDT(-0500)] <js701> JasonElwood looking at https://issues.jasig.org/browse/SSP-819 Journal Comment html. Is there any place in the ui where the comment is displayed as a simple text block. I'm only seeing an editable text field which will show the raw html ok. My understanding from dan, we are not supporting a wysiwyg edit box. Mark ticket as fixed/won't fix?
[17:01:14 CDT(-0500)] <JasonElwood> on Sinclair
[17:04:19 CDT(-0500)] <JasonElwood> emailed MyGPS print example
[17:04:41 CDT(-0500)] <dmccallum541> eagerly awaiting its arrival
[17:05:21 CDT(-0500)] <JasonElwood> it will be there before the clean deploy finishes I bet
[17:13:57 CDT(-0500)] <dmccallum541> so looks like it's a little bit of a slimmed down version of the Action Plan report
[17:14:14 CDT(-0500)] <dmccallum541> basically just tasks
[17:14:45 CDT(-0500)] <JasonElwood> yeah. just tasks
[17:15:32 CDT(-0500)] <dmccallum541> so
[17:15:33 CDT(-0500)] <dmccallum541> ok
[17:15:42 CDT(-0500)] <dmccallum541> that means this is going to be a new jasper report
[17:15:53 CDT(-0500)] <dmccallum541> or
[17:16:04 CDT(-0500)] <dmccallum541> i might be able to whip up a quick JSP
[17:16:21 CDT(-0500)] <JasonElwood> ugh
[17:16:36 CDT(-0500)] <dmccallum541> the JSP, i warn you, would be truly ugly
[17:17:01 CDT(-0500)] <JasonElwood> easier to strip down the action plan report
[17:17:10 CDT(-0500)] <dmccallum541> that's what i'm thinking
[17:17:22 CDT(-0500)] <dmccallum541> i'll have a go at that and let you know if i end up rabbit holing
[17:17:46 CDT(-0500)] <JasonElwood> the anonymous part may be tricky
[17:18:54 CDT(-0500)] <JasonElwood> what about making the print available only to authenticated users and using the action plan report
[17:31:03 CDT(-0500)] <dmccallum541> if print is only available to authenticated users and we're OK with the action plan report, i think this might become trivial