-
Notifications
You must be signed in to change notification settings - Fork 5
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
moira_2.0_support (#2) #3
Conversation
kamaev
commented
Nov 6, 2017
•
edited
Loading
edited
- Added custom headers support
- Added trigger creation by custom id
- Removed tags extra data addition feature
- Changed ttl type from string to int
- Optimized fetch_by_id for large production solutions
* moira_2.0_fixes_0 * moira_2.0_fixes_1 * moira_2.0_fixes_0_doc * moira_2.0_fixes_3 * moira_2.0_fixes_4 * moira_2.0_fixes_5 * moira_2.0_fixes_6 * moira_2.0_fixes_6 * moira_2.0_fixes_7 * moira_2.0_fixes_7 * moira_2.0_fixes_8 * moira_2.0_fixes_9 * moira_2.0_fixes_1_doc * moira_2.0_fixes_10 * moira_2.0_fixes_1_tags * moira_2.0_fixes_2_doc * moira_2.0_fixes_3_doc * moira_2.0_fixes_4_doc
better example for id field usage
Ready for review |
@@ -40,7 +40,7 @@ def __init__(self, api_url, login=None, auth_user=None, auth_pass=None): | |||
if auth_user and auth_pass: | |||
self.auth = HTTPBasicAuth(auth_user, auth_pass) | |||
|
|||
self.auth_header = {'X-Webauth-User': login} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
А какая причина у этого изменения? У нас же захардкожено в API имя этого заголовка, его нельзя поменять.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Исправил
moira_client/models/trigger.py
Outdated
:param ttl_state: str state after ttl seconds without data (one of STATE_* constants) | ||
:param sched: dict schedule for trigger | ||
:param expression: str python expression | ||
:param expression: str golang expression |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Тут C-like expression лучше написать.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Исправил
moira_client/models/trigger.py
Outdated
:param ttl_state: str state after ttl seconds without data (one of STATE_* constants) | ||
:param sched: dict schedule for trigger | ||
:param expression: str python expression | ||
:param expression: str golang expression |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Тут тоже.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Исправил
Add editor settings