icees-kg KP is missing a urlRemote and therefore returns 500 from ARS

[maximusunc]

It's been noticed that icees-kg is always returning a 500 error from queries to the ARS. I did some investigation and noticed that in all ARS environments, /ars/api/actors shows that icees-kg urlRemote is null, whereas other KPs/ARAs have this field set. I'm not entirely sure how the ARS KP registration works, so please correct me if I'm wrong. Do we need to set a urlRemote in order for the ARS to successfully query icees-kg?

@karafecho

[maximusunc]

One other thing I noticed was that there were the other non-TRAPI icees instances still listed. kp-icees-kg is the only one that the ARS needs to know about, kp-icees, kp-icees-pcd, and kp-icees-dili should all be removed.

[ShervinAbd92]

hi @maximusunc, the issue was coming from an extra space between infores and icees-kg in the app folder. once i removed the extra space, urlRemote was returning the url . As for the other icees, even though they are listed, they have been deactivated (active=False) but we can completely wipe them out if you guys prefer that?

[maximusunc]

Hey @ShervinAbd92 I think we would prefer them removed, just so it's clear that they shouldn't be used in Translator. Thanks!

[ShervinAbd92]

sure @maximusunc. we can take care of that :)

[ShervinAbd92]

hey @maximusunc, Mark William just reminded me that the reason we just deactivated them instead of removing them is that all previous query responses from these icees are going to be deleted from our records. i am not sure, if this is what we want?

[maximusunc]

@karafecho what do you think?

[karafecho]

@ShervinAbd92 : Max and I discussed this, and we agree that you should go ahead and remove all prior icees instances EXCEPT icees-kg. Thanks!