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.

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

[Feature] Don't Persist Inflator/Indexer Containers #14

Open
techman83 opened this issue Sep 13, 2019 · 5 comments
Open

[Feature] Don't Persist Inflator/Indexer Containers #14

techman83 opened this issue Sep 13, 2019 · 5 comments
Labels
Enhancement New feature or request Indexer Receives inflated modules and adds them to CKAN-meta Inflator Receives netkans and generates ckans

Comments

@techman83
Copy link
Member

The Inflator + Indexer often sit around doing nothing, querying the SQS queues every 20 seconds generates for both services potentially 6,220,800 messages a month. Whilst that's only a few dollars (and we're saving more than this by only needing a micro vs a small instance), we can do better.

Suggestions

  • Indexer/Inflator after X consecutive time without receiving a message terminate (configurable with an optional flag. Run forever or --exit-after-mins 10)
  • Drop the Service Definitions for the Indexer/Inflator
  • Scheduler will call 'run-task' on the indexer/inflator when a run is started
@HebaruSan
Copy link
Member

Should we consider setting ReceiveMessageRequest.WaitTimeSeconds to a value higher than 20? I don't see a mention of an upper limit or a downside to high values in the documentation, so it looks like it ought to work:

@techman83
Copy link
Member Author

Unfortunately 20 seconds is the max wait time. But I'm really liking the idea of only spinning up the containers when we need to. It's obnoxiously fast to do it this way and I have a bunch of ideas around doing stuff like this at work now.

@DasSkelett DasSkelett added Enhancement New feature or request Indexer Receives inflated modules and adds them to CKAN-meta Inflator Receives netkans and generates ckans labels Oct 23, 2019
@HebaruSan
Copy link
Member

Scheduler will call 'run-task' on the indexer/inflator when a run is started

Would the webhooks also need to do that?

@techman83
Copy link
Member Author

Whatever path we take, it would be a generic call any of the services could call to make it happen.

@techman83
Copy link
Member Author

I should note I was several orders of magnitude off here. We're around 400,000 messages while running the indexer/inflator full time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request Indexer Receives inflated modules and adds them to CKAN-meta Inflator Receives netkans and generates ckans
Projects
None yet
Development

No branches or pull requests

3 participants