Skip to content
This repository was archived by the owner on Mar 4, 2021. It is now read-only.

Add a URL target to add events through HTTP GET; more Calendar logging#223

Merged
ebukoski merged 3 commits into
Netflix:masterfrom
ebukoski:master
Dec 3, 2015
Merged

Add a URL target to add events through HTTP GET; more Calendar logging#223
ebukoski merged 3 commits into
Netflix:masterfrom
ebukoski:master

Conversation

@ebukoski
Copy link
Copy Markdown
Contributor

MonkeyCalendar: new constructor and better logging. Add a constructor to pass in configuration along with open/close/timezone. More logging in isMonkeyTime so it clearer why Monkey is taking time off.

JanitorMonkeyResource: Add URL target to allow adding events with a GET call, this is convenient for opt-in/opt-out through email.

… to pass in configuration along with open/close/timezone. More logging in isMonkeyTime so it clearer why Monkey is taking time off.
@cloudbees-pull-request-builder
Copy link
Copy Markdown

NetflixOSS » SimianArmy » SimianArmy-pull-requests #37 SUCCESS
This pull request looks good

@cloudbees-pull-request-builder
Copy link
Copy Markdown

SimianArmy-pull-requests #209 SUCCESS
This pull request looks good

…ing. Add try-catch in janitor loop so one janitor failure doesn't stop the whole monkey run. AbstractJanitor: add checkedResourcesCount as a servo metric. Fix wrong resource passed into addFieldsandTagsToEvent() method.
@cloudbees-pull-request-builder
Copy link
Copy Markdown

NetflixOSS » SimianArmy » SimianArmy-pull-requests #38 SUCCESS
This pull request looks good

@cloudbees-pull-request-builder
Copy link
Copy Markdown

SimianArmy-pull-requests #210 SUCCESS
This pull request looks good

ebukoski added a commit that referenced this pull request Dec 3, 2015
Add a URL target to add events through HTTP GET; more Calendar logging
@ebukoski ebukoski merged commit 2b16f66 into Netflix:master Dec 3, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants