Skip to content
This repository has been archived by the owner on May 22, 2024. It is now read-only.

Update Elasticsearch to version 5.6 or remove service if it's not being used - eregs #396

Closed
3 tasks done
Tracked by #139
lbeaufort opened this issue Apr 17, 2018 · 14 comments
Closed
3 tasks done
Tracked by #139
Assignees
Milestone

Comments

@lbeaufort
Copy link
Member

lbeaufort commented Apr 17, 2018

cloud.gov now has support for Elasticsearch version 5.6 more info here, which may improve some of the performance issues we've been having with our current version (2.4).

Also, cloud.gov may end support for 2.4 in the medium-term future.

See openFEC work: fecgov/openFEC#2921
Thank you @fec-jli for flagging this!

Completion criteria:

  • Remove Elasticsearch 2.4 services from dev instance and test to make sure things still work

    • cf delete-service fec-api-search
    • cf unbind-service eregs fec-eregs-search
    • cf delete-service fec-eregs-search
  • Remove from stage and test there

    • cf unbind-service eregs fec-eregs-search
    • cf delete-service fec-eregs-search
    • fec-api-search : This ES24 service is already deleted from stage space .No action required
  • Remove from production and test again

    • cf unbind-service eregs fec-eregs-search
    • cf delete-service fec-eregs-search
    • fec-api-search : This ES24 service is already deleted from prod space .No action required
@lbeaufort lbeaufort changed the title Update elasticsearch to version 5.6 - eregs Update Elasticsearch to version 5.6 - eregs Apr 17, 2018
@lbeaufort lbeaufort added this to the Sprint 5.5 milestone Apr 17, 2018
@pkfec
Copy link
Contributor

pkfec commented Apr 17, 2018

By looking at the fec-eregs repo requirements.txt we have pyelasticsearch==1.4 package. eregs repo doesn't have a direct reference to latest Elastic Search library.

I do not think this repo needs any updates.

@lbeaufort
Copy link
Member Author

lbeaufort commented Apr 17, 2018

@pkfec thanks for taking a look! If you look at cf services it looks like eregs has an instance of elasticsearch set up that's version 2.4 called fec-eregs-search... I'm not sure how/if they work together and whether it matters if eregs is using Elasticsearch 2.4 - @vrajmohan thoughts?

@AmyKort AmyKort removed this from the Sprint 5.5 milestone Apr 17, 2018
@pkfec
Copy link
Contributor

pkfec commented Apr 18, 2018

i think its a matter of creating a new ES cloud service elasticsearch56 in cloud and binding it the fec-eregs-search.

cc @vrajmohan and @lbeaufort

@vrajmohan
Copy link
Contributor

@pkfec is correct, but I am curious to see if the service is needed at all as we use fec-api-search for searching regulations as well.

@AmyKort
Copy link

AmyKort commented Jun 6, 2018

What is the status if this issue?

@lbeaufort
Copy link
Member Author

@AmyKort This issue hasn't been assigned or given a milestone.

@AmyKort
Copy link

AmyKort commented Jun 6, 2018

@lbeaufort I can see that. Nevertheless, folks have been poking at it, and it looks like we ended at a cliff hanger question about whether we need this service at all. So, I'm curious if there's an answer to that question that's not recorded here.

@lbeaufort lbeaufort changed the title Update Elasticsearch to version 5.6 - eregs Update Elasticsearch to version 5.6 or remove service if it's not being used - eregs Jun 6, 2018
@lbeaufort
Copy link
Member Author

@AmyKort I updated the title to reflect the additional work - I would want to do careful research to make sure it could get removed.

@pkfec
Copy link
Contributor

pkfec commented Oct 17, 2018

Annual regulations are saved in AWS shared database instance in the cloud. eregs app do not consume the elasticsearch service at all. I have deleted below two es24 services from dev space that are no longer needed or accessed from our apps anywhere.

  1. fec-api-search
  2. fec-eregs-search

Also, removed the fec-eregs-search service from dev/stage/prod manifest files.

@pkfec
Copy link
Contributor

pkfec commented Oct 17, 2018

@AmyKort @dorothyyeager i have updated the completion criteria a little bit.

@AmyKort
Copy link

AmyKort commented Oct 17, 2018

Awesome! Thanks @pkfec !

@dorothyyeager
Copy link
Contributor

Yes, thanks!

@JonellaCulmer JonellaCulmer modified the milestones: Sprint 7.4, Sprint 7.5 Oct 30, 2018
@pkfec
Copy link
Contributor

pkfec commented Oct 31, 2018

Deleted the unused ES24 service fec-eregs-search from Stage space today. On Prodspace, i will delete the fec-eregs-search after the 7.4 release( scheduled on 11/6/2018).

@pkfec
Copy link
Contributor

pkfec commented Nov 6, 2018

I have deleted the unused fec-eregs-search ES24 service from prod space after the 7.4 release today.
Closing this issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants