Update pipeline v2 spec

Author: Itai Gendler

docs/content/pipelines v2/spec.md

@@ -12,9 +12,9 @@ A Pipeline also needs a `.spec` section.
 #### Pipeline which is stored entirely in Codefresh
 ```yaml
 apiVersion: "v1"
-kind: "context"
-name: "new-pipeline"
+kind: "pipeline"
 metadata:
+  name: "new-pipeline"
   labels:
     repo: "github:ArikMaor/ping-server"
 spec:
@@ -44,6 +44,7 @@ spec:
 #### Pipeline which is stored on a remote git
 ```yaml
 apiVersion: "v1"
+kind: "pipeline"
 metadata:
   name: "ew-pipeline-git"
   labels:
@@ -71,6 +72,7 @@ spec:
 #### Pipeline which is stored on a specific url
 ```yaml
 apiVersion: "v1"
+kind: "pipeline"
 metadata:
   name: "new-pipeline-url"
   labels:

docs/index.js

@@ -10,6 +10,7 @@ const TEMP_DIR = path.resolve(__dirname, '../temp');
 const TEMPLATE_DIR = path.resolve(__dirname);
 const FILES_TO_IGNORE = ['index.js', 'content/pipelines v2/_index.md', 'content/pipelines v2/spec.md'];
 const baseDir = path.resolve(TEMP_DIR, './content');
+const ALLOW_BETA_COMMANDS = process.env.ALLOW_BETA_COMMANDS;
 
 
 const deleteTempFolder = async () => {
@@ -190,7 +191,7 @@ const createAutomatedDocs = async () => {
         const command = require(file);
 
         // dont document beta commands currently
-        if (command.isBetaCommand()) {
+        if (command.isBetaCommand() && !ALLOW_BETA_COMMANDS) {
             continue;
         }
 

lib/interface/cli/commands/pipeline2/create.cmd.js

@@ -23,23 +23,7 @@ const command = new Command({
             });
     },
     handler: async (argv) => {
-        const {filename, name} = argv;
-
-        if (!filename) {
-            throw new CFError('Pipeline definitions file must be provided');
-        }
-
-        if (!_.get(filename, 'metadata.name') && !name) {
-            throw new CFError('Name must be provided');
-        }
-
-        const data = argv.filename;
-        if (name) {
-            _.set(data, 'metadata.name', name);
-        }
-
-        await pipeline2.createPipeline(data);
-        console.log(`Pipeline '${data.metadata.name}' created`);
+        throw new CFError('Not implemented');
     },
 });
 

lib/interface/cli/commands/root/create.cmd.js

@@ -1,7 +1,7 @@
 const CFError = require('cf-errors');
 const Command = require('../../Command');
 const { crudFilenameOption } = require('../../helpers/general');
-const { context } = require('../../../../logic').api;
+const { context, pipeline2 } = require('../../../../logic').api;
 const yargs = require('yargs');
 
 const get = new Command({
@@ -31,6 +31,10 @@ const get = new Command({
                 await context.createContext(data);
                 console.log(`Context: ${data.metadata.name} created`);
                 break;
+            case 'pipeline':
+                await pipeline2.createPipeline(data);
+                console.log(`Pipeline '${data.metadata.name}' created`);
+                break;
             default:
                 throw new CFError(`Entity: ${entity} not supported`);
         }

package.json

@@ -10,6 +10,7 @@
     "ci-publish": "ci-publish",
     "pkg": "pkg . -t node9-macos-x64,node9-linux-x64,node9-win-x64 --out-path ./dist",
     "serve-docs": "yarn build-local-docs && cd temp && hugo server -D",
+    "serve-docs-beta": "ALLOW_BETA_COMMANDS=true yarn build-local-docs && cd temp && hugo server -D",
     "build-local-docs": "node ./docs/index.js",
     "build-public-docs": "node ./docs/index.js && cd temp && hugo"
   },