search

OperationCode / operationcode_infra

Platform infrastructure for the Operation Code site.
MIT License
15 stars 14 forks source link

Pybot service #86

Closed apex-omontgomery closed 5 years ago

apex-omontgomery commented 5 years ago

Add's pybot service url to the pods using the expected values.

Will also need to figure out how to add the dns values for staging.

ohaiwalt commented 5 years ago

Could you add comments on these with the repo link, or some additional information on what they are? I also need to do that for the dashboards record. It'll help when we go to determine whether a record is necessary any longer.

Comments can be added like so:

// single line comment

/*
multi
line
comment
*/
ohaiwalt commented 5 years ago

Also, question: Does sidekiq need the Slack var?

apex-omontgomery commented 5 years ago

What do you mean

ohaiwalt commented 5 years ago

What do you mean

Is sidekiq sending messages to Slack? Curiosity question.

apex-omontgomery commented 5 years ago

Currently sidekiq is supposed to send directly to slack api, in the near future it's going to go through our slack service (pybot) and at that point we can remove some of these slack credentials.

ohaiwalt commented 5 years ago

Is this ready to go? If so, I can merge after a rebase.

apex-omontgomery commented 5 years ago

No, I need the correct urls it should resolve to. That's on my list to get with allen. I'm not sure how ELB and how it's setup is going to affect this.

apex-omontgomery commented 5 years ago

Not exactly ready to merge. I still don't know the correct urls for both staging and prod with the pybot instance. I'll try and update today.