@f5devcentral/f5-cloud-libs - npm Package Compare versions

Comparing version 4.2.0-beta.1 to 4.2.0

lib/autoscaleProvider.js

@@ -406,8 +406,7 @@ /**
  * @param {String} instancId - Instance ID that was elected master.
- * @param {Object} instances - Dictionary of instances as returned by getInstances.
  *
  * @returns {Promise} A promise which will be resolved when processing is complete.
  */
-AutoscaleProvider.prototype.masterElected = function masterElected(instanceId, instances) {
-    this.logger.debug('No override for AutoscaleProvider.masterElected', instanceId, instances);
+AutoscaleProvider.prototype.masterElected = function masterElected(instanceId) {
+    this.logger.debug('No override for AutoscaleProvider.masterElected', instanceId);
     return q();
@@ -485,3 +484,3 @@ };
  *
- * @param {String} actionId                 - Action id of message to send
+ * @param {String} action                   - Action id of message to send
  * @param {Object} [options]                - Optional parameters
@@ -495,5 +494,5 @@ * @param {String} [options.toInstanceId]   - Instance ID that message is for
  */
-AutoscaleProvider.prototype.sendMessage = function sendMessage(actionId, data) {
+AutoscaleProvider.prototype.sendMessage = function sendMessage(action, data) {
     if (this.hasFeature(AutoscaleProvider.FEATURE_MESSAGING)) {
-        throw new Error('Unimplemented abstract method AutoscaleProvider.sendMessage', actionId, data);
+        throw new Error('Unimplemented abstract method AutoscaleProvider.sendMessage', action, data);
     } else {
@@ -500,0 +499,0 @@ this.logger.debug('No override for AutoscaleProvider.sendMessage');

lib/bigIp.js

@@ -986,2 +986,26 @@ /**
         })
+        .then(() => {
+            // the UCS file can take a while to show up...
+            const fullPath = `/var/local/ucs/${file}.ucs`;
+            const checkFile = function () {
+                const deferred = q.defer();
+                fs.access(fullPath, (err) => {
+                    if (err) {
+                        deferred.reject();
+                    } else {
+                        deferred.resolve();
+                    }
+                });
+                return deferred.promise;
+            };
+
+            return util.tryUntil(
+                this,
+                {
+                    maxRetries: 60,
+                    retryIntervalMs: 2000
+                },
+                checkFile
+            );
+        })
         .catch((err) => {
@@ -988,0 +1012,0 @@ this.logger.info('saveUcs failed', err);

lib/httpUtil.js

@@ -161,3 +161,5 @@ /**
                     if (statusCode >= 300) {
-                        const message = `${url.toString()} returned with status code ${statusCode}`;
+                        /* eslint-disable max-len */
+                        const message = `${url.toString()} status code ${statusCode}, status message ${response.statusMessage}`;
+                        /* eslint-enable max-len */
                         deferred.reject(new Error(message));
@@ -164,0 +166,0 @@ response.resume();

package.json

 {
   "name": "@f5devcentral/f5-cloud-libs",
-  "version": "4.2.0-beta.1",
+  "version": "4.2.0",
   "description": "Common library code and scripts for deploying a BIG-IP in a cloud environment",
@@ -5,0 +5,0 @@ "keywords": [

RELEASE_NOTES.md

@@ -6,2 +6,3 @@ # Release notes
 * Support for reading from rest storage
+* Fix for vlan name in 1 nic configurations
 
@@ -8,0 +9,0 @@ ## Version 4.1.0

scripts/cluster.js

@@ -310,3 +310,7 @@ /**
                             logger.info('Storing credentials.');
-                            return provider.putMasterCredentials();
+                            return util.tryUntil(
+                                provider,
+                                util.DEFAULT_RETRY,
+                                provider.putMasterCredentials
+                            );
                         }
@@ -322,3 +326,8 @@ return q();
                             logger.info('Getting master credentials.');
-                            return provider.getMasterCredentials(options.remoteHost, options.remotePort);
+                            return util.tryUntil(
+                                provider,
+                                util.DEFAULT_RETRY,
+                                provider.getMasterCredentials,
+                                [options.remoteHost, options.remotePort]
+                            );
                         }
@@ -325,0 +334,0 @@ return q();

scripts/commonOptions.js

@@ -36,3 +36,3 @@ /**
         return options
-            .version('4.2.0-beta.1')
+            .version('4.2.0')
             .option(
@@ -39,0 +39,0 @@ '--host <ip_address>',

scripts/decryptDataFromFile.js

@@ -43,3 +43,3 @@ /**
                 options
-                    .version('4.2.0-beta.1')
+                    .version('4.2.0')
                     .option('--data-file <data_file>', 'Full path to file with data (use this or --data)')
@@ -46,0 +46,0 @@ .parse(argv);

scripts/encryptDataToFile.js

@@ -58,3 +58,3 @@ /**
                 options
-                    .version('4.2.0-beta.1')
+                    .version('4.2.0')
                     .option(
@@ -61,0 +61,0 @@ '--background',

scripts/generatePassword.js

@@ -36,3 +36,3 @@ /**
             options
-                .version('4.2.0-beta.1')
+                .version('4.2.0')
                 .option(
@@ -39,0 +39,0 @@ '--length <password_length>',

scripts/network.js

@@ -65,3 +65,3 @@ /**
                 options
-                    .version('4.2.0-beta.1')
+                    .version('4.2.0')
                     .option(
@@ -68,0 +68,0 @@ '--host <ip_address>',

scripts/runScript.js

@@ -51,3 +51,3 @@ /**
                 options
-                    .version('4.2.0-beta.1')
+                    .version('4.2.0')
                     .option(
@@ -54,0 +54,0 @@ '--background',

test/lib/bigIpTests.js

@@ -25,2 +25,3 @@ /**
 const realExecFile = childProcessMock.execFile;
+const realAccess = fs.access;
 
@@ -79,2 +80,3 @@ const decryptedPassword = 'foofoobarbar';
         fs.unlink = realUnlink;
+        fs.access = realAccess;
         childProcessMock.execFile = realExecFile;
@@ -1458,2 +1460,6 @@
 
+            fs.access = function(file, cb) {
+                cb();
+            };
+
             setTimeout = function(cb) {
@@ -1460,0 +1466,0 @@ cb();

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

No v1

Quality

Package is not semver >=1. This means it is not stable and does not support ^ ranges.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

1192832

increased by0.21%

Lines of code

28589

increased by0.19%

Number of low quality alerts

1

decreased by-50%

No dependency changes