update job create to support dockerfile logic

Author: exdx

lib/jobs/create.js

@@ -21,7 +21,7 @@ const chalk = require('chalk');
  * @method create
  * @description Create a new Paperspace job, and tail its log output if run at the command line. To disable the tailing behavior specify '--tail false'.  Note: if a project is not defined for the current working directory, and you are running in command line mode, a project configuration settings file will be created.  Use '--init false' or specify '--project [projectname]' to override this behavior.
  * @param {object} params - Job creation parameters
- * @param {string} params.container - A required reference to a docker image in a public or private docker registry, or a container name provided by Paperspace.  Docker image repository references must be in lowercase and may include a tag and a hostname prefix followed by a slash; if ommitted the hostname defaults to that of the public Docker Hub registry.  An example docker image reference: 'docker.io/mynamespace/myimage:mytag'.  A container name may be mixed case.  (Designated container names are currently only provided as part of various Gradient tutorials and samples.)
+ * @param {string} params.container - A reference to a docker image in a public or private docker registry, or a container name provided by Paperspace.  Docker image repository references must be in lowercase and may include a tag and a hostname prefix followed by a slash; if ommitted the hostname defaults to that of the public Docker Hub registry.  An example docker image reference: 'docker.io/mynamespace/myimage:mytag'.  A container name may be mixed case.  (Designated container names are currently only provided as part of various Gradient tutorials and samples.)
  * @param {string} [params.cluster] - An optional cluster name of a cluster to run the job on. Only one of cluster or clusterId may be specified.
  * @param {string} [params.clusterId] - An optional cluster id of a cluster to run the job on. Only one of cluster or clusterId may be specified.
  * @param {string} [params.machineType] - An optional machine type to run the job on: either 'GPU+', 'P4000', 'P5000', 'P6000', 'V100', 'K80', 'P100', 'TPU', or 'GradientNode'.<p>Defaults to 'K80'. <P>Note: the 'K80', 'P100', and 'TPU' machineTypes run on Google Cloud Platform (GCP).  The other machineTypes run on the Paperspace Cloud.  Google Cloud platform and Paperspace Cloud have distict Job Storage spaces;  Job storage is not currently shared between these two cloud environments.
@@ -196,7 +196,11 @@ function create(params, cb) {
 		delete params.tail;
 	}
 
-	params.command = Buffer.from(params.command).toString('base64');
+	if (params.useDockerfile && params.buildOnly) {
+		params.command = null
+	} else {
+		params.command = Buffer.from(params.command).toString('base64');
+	}
 
 	// XXX TODO trim leading/trailing spaces from input paths
 	// XXX TODO whitelist git services
@@ -239,7 +243,7 @@ function create(params, cb) {
 
 	projectConfig.setMachineType(params.project, params.machineType);
 
-	projectConfig.setContainer(params.project, params.container);
+	projectConfig.setContainer(params.project, params.container, params.useDockerfile);
 
 	projectConfig.setCommand(params.project, params.command);
 
@@ -249,7 +253,7 @@ function create(params, cb) {
 
 	projectConfig.setCluster(params.project, params.cluster);
 
-	if (!params.container) return ifCliPrintErrorOnly(new Error('Missing required parameter `container`'));
+	if (!params.container && !params.useDockerfile) return ifCliPrintErrorOnly(new Error('Missing required parameter `container`'));
 
 	if (!params.workspace) params.workspace = cwd;
 
@@ -516,9 +520,6 @@ assign(create, {
 	name: 'create',
 	method: 'post',
 	route: '/jobs/createJob',
-	requires: {
-		container: 'string',
-	},
 	file: 'workspace',
 	returns: {},
 });

lib/projectConfig.js

@@ -74,7 +74,10 @@ function getContainer(project, container) {
 	return container;
 }
 
-function setContainer(project, container) {
+function setContainer(project, container, useDockerfile) {
+	if (useDockerfile) {
+		return
+	}
 	if (container) {
 		if ((!project || config.project === project) && config.container !== container) {
 			config.container = container;