agrc / porter

UGRC tracks the additions, replacements, and deletions of SGID items (in the broadest sense of add, replace, or delete) through issues in this repository.
https://gis.utah.gov/documentation/policy/
MIT License
2 stars 0 forks source link

Remove Geolocators from mapserv #102

Closed stdavis closed 1 year ago

stdavis commented 3 years ago

Summary

http://mapserv.utah.gov/arcgis/rest/services/Geolocators

Mapserv has long hosted our geocoders in this folder. I wonder if they were originally used to back the web API geocoder? Now-a-days, the web API has it's own, dedicated ArcGIS Server instance and does not hit mapserv. No one has been updating these locators so they are likely years out-of-date. I propose that we deprecate them in favor of the web API or the downloadable locators on our website that Greg keeps up-to-date.

There is likely a bit of research that needs to be done to see what, if anything, is using these.

Triage

Remove data from the following areas

~- [ ] Internal SGID (assigned to)~ ~- [ ] SGID10 (assigned to)~

Update information in the following areas

Is there a map service?

Are there service dependencies?

Notification

steveoh commented 3 years ago

My only concern is arcgis only clients that cannot access the web api

stdavis commented 3 years ago

Great point. I forgot to mention that we may want to wait until masquerade is ready for prime time.

steveoh commented 3 years ago

Let's block this until masquerade is available.

In the mean time should we start to maintain that locator?

steveoh commented 3 years ago

These are actually being maintained by forklift. Should we continue to provide them?

stdavis commented 3 years ago

I feel like that masquerade will give much better results overall once it's up-to-snuff so we should deprecate these at that point.

stdavis commented 1 year ago

The next step for this might be to check the (perhaps unreliable?) ArcGIS Server usage stats for these services.

agrc-conductor commented 1 year ago

conductor results for tasks - 102

check status
@gregbunce has completed 1 out of 3 tasks :no_entry:
@steveoh has completed 0 out of 2 tasks :no_entry:
@rkelson has completed 0 out of 1 tasks :no_entry:
stdavis commented 1 year ago

App1: image

App2: image

It does look like these get some traffic. However, it doesn't look like folks are running large geocoding jobs. I do think that these are low enough that we should still work to deprecate these services.

agrc-conductor commented 1 year ago

conductor results for tasks - 102

check status
@gregbunce has completed 1 out of 3 tasks :no_entry:
@steveoh has completed 0 out of 2 tasks :no_entry:
@rkelson has completed 0 out of 1 tasks :no_entry:
agrc-conductor commented 1 year ago

conductor results for tasks - 102

check status
@gregbunce has completed 1 out of 3 tasks :no_entry:
steveoh has completed 4 out of 4 tasks :+1:
rkelson has completed 1 out of 1 tasks :+1:
@stdavis has completed 0 out of 3 tasks :no_entry:
stdavis commented 1 year ago

What is the appropriate messaging for this deprecation? Tweet + email? Here is some possible verbiage:

Heads up! We are deprecating our geolocator services "AddressPoints_AddressSystem" and "Roads_AddressSystem_STREET" on mapserv.utah.gov in favor of Masquerade (https://github.com/agrc/masquerade#readme) on Monday, December 5th. Please upgrade any of your projects before then.

@steveoh Can you help with getting this info out there? Does 4 weeks seem like a reasonable timeframe?

steveoh commented 1 year ago

What email addresses would you use?

I'm fine to proceed without notification but a tweet seems normal and your message is good.

@brigsz, when is the next newsletter?

brigsz commented 1 year ago

I don't think it is until January, around the week of the 17th is what I have on the calendar

stdavis commented 1 year ago

I'm good with just a tweet. I'm not sure that the usage is enough to justify a separate spam to our newsletter email list.

steveoh commented 1 year ago

/remind someone to remove the locators on december 5th

github-actions[bot] commented 1 year ago

@steveoh set a reminder for 12/5/2022

github-actions[bot] commented 1 year ago

:wave: @someone, remove the locators

agrc-conductor commented 1 year ago

conductor results for tasks - 102

check status
@gregbunce has completed 1 out of 3 tasks :no_entry:
steveoh has completed 4 out of 4 tasks :+1:
rkelson has completed 1 out of 1 tasks :+1:
@stdavis has completed 0 out of 3 tasks :no_entry:
stdavis commented 1 year ago

I stopped the locators on both machines today.

/remind me to delete the locators in two weeks

github-actions[bot] commented 1 year ago

@stdavis set a reminder for 12/23/2022

steveoh commented 1 year ago

I only saw a handful of failed requests in the logs. Hopefully everyone has migrated.

gregbunce commented 1 year ago

i'm reading that i can remove the reference to desktop locators on our website now. is that correct? sounds like we've made the appropriate notices and announcements. just want to confirm before i do so.

stdavis commented 1 year ago

Nick K from Tax chatted me this morning about the shut down of the address point locator. He had a webmap wired up to it. I sent him the links for masquerade and he got it switched over.

gregbunce commented 1 year ago

verified that there were no references to the arcgis 10x locators/geocoders in the stewardship sheet and/or sgid index

gregbunce commented 1 year ago

PR 2174 removes references to the arcgis 10x locators from gis.utah.gov

steveoh commented 1 year ago

@stdavis will you merge the above pr when you remove the locators on the 23rd? I won't be working then.

github-actions[bot] commented 1 year ago

:wave: @stdavis, delete the locators

stdavis commented 1 year ago

Locators have been removed from mapserv and the associated gis.utah.gov PR has been merged.

gregbunce commented 1 year ago

thanks for checking the box for me, @stdavis. you beat me to it. gis.utah.gov data pages (search for links on our website)