Add release job for demo functions (#85)

* Add release job for demo functions

* Remove globall install from docs

* format

Author: frankfarzan

.github/workflows/release-create-kpt-functions.yaml

@@ -6,7 +6,7 @@ on:
       - release-create-kpt-functions-*
 
 jobs:
-  node-ci:
+  release:
     runs-on: ubuntu-latest
     env:
       GOPATH: /home/runner/work/kpt-functions-sdk/go

.github/workflows/release-demo-functions.yaml

@@ -0,0 +1,45 @@
+name: release-create-kpt-functions
+
+on:
+  push:
+    tags:
+      - release-demo-functions-*
+
+jobs:
+  e2e-ci:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        node-version: [12.x]
+    steps:
+    - uses: actions/checkout@v1
+    - name: Set up gcloud
+      uses: GoogleCloudPlatform/github-actions/setup-gcloud@master
+      with:
+          version: '275.0.0'
+          service_account_email: ${{ secrets.GCP_SA_EMAIL }}
+          service_account_key: ${{ secrets.GCP_SA_KEY }}
+    - name: Use Node.js ${{ matrix.node-version }}
+      uses: actions/setup-node@v1
+      with:
+        node-version: ${{ matrix.node-version }}
+        registry-url: 'https://npm.pkg.github.com'
+    - name: Install NPM packages
+      run: |
+        cd ts/demo-functions
+        npm ci
+        envsubst < $NPM_CONFIG_USERCONFIG  > .npmrc
+      env:
+        NODE_AUTH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+    - name: Build docker images
+      run: |
+        cd ts/demo-functions
+        npm run kpt:docker-build -- --tag=latest
+    - name: Run e2e tests
+      run: |
+        npm install -g bats
+        TAG=latest bats tests/e2e.bats
+    - name: Push docker images
+      run: |
+        cd ts/demo-functions
+        npm run kpt:docker-push -- --tag=latest

.github/workflows/release-kpt-functions.yaml

@@ -6,7 +6,7 @@ on:
       - release-kpt-functions-*
 
 jobs:
-  node-ci:
+  release:
     runs-on: ubuntu-latest
     steps:
     - uses: actions/checkout@v1

docs/develop.md

@@ -128,13 +128,6 @@ Run the interactive initializer:
 npm init @googlecontainertools/kpt-functions
 ```
 
-Or, alternatively install the package globally and then run it:
-
-```sh
-npm install -g @googlecontainertools/create-kpt-functions
-create-kpt-functions
-```
-
 Follow the instructions and respond to all prompts.
 
 This process will create the following:

tests/e2e.bats

@@ -1,7 +1,7 @@
 #!/usr/bin/env bats
 
+export TAG=${TAG:-dev}
 export EXAMPLE_CONFIGS=${BATS_TEST_DIRNAME}/../example-configs
-export TAG=dev
 export EMPTY_OUTPUT=$(cat <<-EOF
 apiVersion: v1
 kind: List

ts/demo-functions/src/label_namespace.ts

@@ -0,0 +1,48 @@
+/**
+ * Copyright 2019 Google LLC
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+import { addLabel, Configs } from '@googlecontainertools/kpt-functions';
+import { isNamespace } from './gen/io.k8s.api.core.v1';
+
+export const LABEL_NAME = 'label_name';
+export const LABEL_VALUE = 'label_value';
+
+export async function labelNamespace(configs: Configs) {
+  const labelName = configs.getFunctionConfigValueOrThrow(LABEL_NAME);
+  const labelValue = configs.getFunctionConfigValueOrThrow(LABEL_VALUE);
+  configs.get(isNamespace).forEach((n) => addLabel(n, labelName, labelValue));
+}
+
+labelNamespace.usage = `
+Adds a label to all Namespaces.
+
+Configured using a ConfigMap with the following keys:
+
+${LABEL_NAME}: Label name to add to Namespaces.
+${LABEL_VALUE}: Label value to add to Namespaces.
+
+Example:
+
+To add a label 'color: orange' to Namespaces:
+
+apiVersion: v1
+kind: ConfigMap
+data:
+  ${LABEL_NAME}: color
+  ${LABEL_VALUE}: orange
+metadata:
+  name: my-config
+`;

ts/demo-functions/src/label_namespace_run.ts

@@ -0,0 +1,20 @@
+/**
+ * Copyright 2019 Google LLC
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+import { labelNamespace } from './label_namespace';
+import { run } from '@googlecontainertools/kpt-functions';
+
+run(labelNamespace);

ts/demo-functions/src/label_namespace_test.ts

@@ -0,0 +1,80 @@
+/**
+ * Copyright 2019 Google LLC
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+import { Configs, TestRunner } from '@googlecontainertools/kpt-functions';
+import { labelNamespace, LABEL_NAME, LABEL_VALUE } from './label_namespace';
+import { Namespace, ConfigMap } from './gen/io.k8s.api.core.v1';
+
+const TEST_NAMESPACE = 'testNamespace';
+const TEST_LABEL_NAME = 'costCenter';
+const TEST_LABEL_VALUE = 'xyz';
+
+describe('labelNamespace', () => {
+  let functionConfig = ConfigMap.named('foo');
+  functionConfig.data = {};
+  functionConfig.data[LABEL_NAME] = TEST_LABEL_NAME;
+  functionConfig.data[LABEL_VALUE] = TEST_LABEL_VALUE;
+
+  const RUNNER = new TestRunner(labelNamespace);
+
+  it('empty input ok', RUNNER.assertCallback(new Configs(undefined, functionConfig)));
+
+  it('requires functionConfig', RUNNER.assertCallback(undefined, undefined, TypeError));
+
+  it('adds label namespace when metadata.labels is undefined', async () => {
+    const actual = new Configs(undefined, functionConfig);
+    actual.insert(Namespace.named(TEST_NAMESPACE));
+
+    const expected = new Configs();
+    expected.insert(
+      new Namespace({
+        metadata: {
+          name: TEST_NAMESPACE,
+          labels: { [TEST_LABEL_NAME]: TEST_LABEL_VALUE },
+        },
+      }),
+    );
+
+    await RUNNER.assert(actual, expected);
+  });
+
+  it('adds label to namespace when metadata.labels is defined', async () => {
+    const actual = new Configs(undefined, functionConfig);
+    actual.insert(
+      new Namespace({
+        metadata: {
+          name: TEST_NAMESPACE,
+          labels: { a: 'b' },
+        },
+      }),
+    );
+
+    const expected = new Configs();
+    expected.insert(
+      new Namespace({
+        metadata: {
+          name: TEST_NAMESPACE,
+          labels: {
+            a: 'b',
+            [TEST_LABEL_NAME]: TEST_LABEL_VALUE,
+          },
+        },
+      }),
+    );
+
+    await RUNNER.assert(actual, expected);
+  });
+});

ts/hello-world/src/label_namespace.ts

@@ -23,7 +23,7 @@ export const LABEL_VALUE = 'label_value';
 export async function labelNamespace(configs: Configs) {
   const labelName = configs.getFunctionConfigValueOrThrow(LABEL_NAME);
   const labelValue = configs.getFunctionConfigValueOrThrow(LABEL_VALUE);
-  configs.get(isNamespace).forEach(n => addLabel(n, labelName, labelValue));
+  configs.get(isNamespace).forEach((n) => addLabel(n, labelName, labelValue));
 }
 
 labelNamespace.usage = `

ts/hello-world/src/label_namespace_test.ts

@@ -51,7 +51,7 @@ describe('labelNamespace', () => {
     await RUNNER.assert(actual, expected);
   });
 
-  it('adds label to namespace when metadata.labels is defined', async  () => {
+  it('adds label to namespace when metadata.labels is defined', async () => {
     const actual = new Configs(undefined, functionConfig);
     actual.insert(
       new Namespace({