Update retention policy for bucket if time changes

influxdata · Feb 20, 2019 · 46cd249 · 46cd249
1 parent ec1019b
commit 46cd249
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,7 @@
 1. [11954](https://github.com/influxdata/influxdb/pull/11954): Add the ability to run a task manually from tasks page
 
 ### Bug Fixes
+1. [11997](https://github.com/influxdata/influxdb/pull/11997): Update the bucket retention policy to update the time in seconds
 
 ### UI Improvements
 

diff --git a/ui/src/organizations/components/Retention.tsx b/ui/src/organizations/components/Retention.tsx
@@ -23,6 +23,8 @@ interface Props {
   onChangeRuleType: (type: BucketRetentionRules.TypeEnum) => void
 }
 
+const DEFAULT_SECONDS = 0
+
 export default class Retention extends PureComponent<Props> {
   public render() {
     const {retentionSeconds, type} = this.props
@@ -56,6 +58,7 @@ export default class Retention extends PureComponent<Props> {
   }
 
   private handleRadioClick = (type: BucketRetentionRules.TypeEnum) => {
+    this.props.onChangeRetentionRule(DEFAULT_SECONDS)
     this.props.onChangeRuleType(type)
   }
 

diff --git a/ui/src/organizations/components/UpdateBucketOverlay.tsx b/ui/src/organizations/components/UpdateBucketOverlay.tsx
@@ -75,7 +75,7 @@ export default class BucketOverlay extends PureComponent<Props, State> {
     )
 
     if (!rule) {
-      return 3600
+      return 0
     }
 
     return rule.everySeconds
@@ -104,8 +104,10 @@ export default class BucketOverlay extends PureComponent<Props, State> {
     this.setState({bucket})
   }
 
-  private handleChangeRuleType = ruleType => {
-    this.setState({ruleType})
+  private handleChangeRuleType = async (
+    ruleType: BucketRetentionRules.TypeEnum
+  ) => {
+    await this.setState({ruleType})
   }
 
   private handleSubmit = (e): void => {