chore(pods.log): remove the deprarecated pods.log function. (#196)

BREAKING CHANGE: `pods.log` doesn't follow the kubernetes-client API
conventions. Replace uses of `pods.log('foo')` with `pods('foo').log.get()`.

lib/pods.js

@@ -1,7 +1,5 @@
 'use strict';
 
-const util = require('util');
-
 const BaseObject = require('./base');
 const ContainerBaseObject = require('./container-base-object');
 
@@ -39,24 +37,6 @@ class Pods extends BaseObject {
       }),
       name: options.name || 'pods'
     }));
-
-    this.log = util.deprecate(
-      this.log.bind(this),
-      'pods.log is deprecated and will be removed in 4.0.0. ' +
-      'Use pods(name).log.get instead.');
-  }
-
-  /**
-   * @deprecated Will be removed in 4.0.0. Use pods(name).log.get instead.
-   * @param {RequestOptions|string} options - GET options, or resource name
-   * @param {callback} cb - The callback that handles the response
-   * @returns {Stream} If cb is falsy, return a Stream
-   */
-  log(options, cb) {
-    if (typeof options === 'string') options = { name: options };
-    return this.get(
-      Object.assign({}, options, { name: `${ options.name }/log` }),
-      cb);
   }
 }
 

test/pods.test.js

@@ -173,13 +173,6 @@ describe('lib.pods', () => {
         done();
       });
     });
-    only('unit', 'returns the Pod via the legacy method', done => {
-      common.api.ns(common.currentName).pods.get('test-pod', (err, pod) => {
-        assume(err).is.falsy();
-        assume(pod.kind).is.equal('Pod');
-        done();
-      });
-    });
   });
 
   describe('.delete', () => {
@@ -216,13 +209,6 @@ describe('lib.pods', () => {
         done();
       });
     });
-    only('unit', 'returns log contents via legacy method', done => {
-      common.api.ns(common.currentName).pods.log('test-pod', (err, contents) => {
-        assume(err).is.falsy();
-        assume(contents).is.equal('some log contents');
-        done();
-      });
-    });
   });
 
   common.afterTesting('int', common.cleanupName);