refactor taskfile

Author: diamonwiggins

.github/workflows/mlflow-ci.yml

@@ -55,8 +55,8 @@ jobs:
         working-directory: applications/mlflow
         run: |
           # Use Taskfile to add Helm repos, lint charts and generate templates
-          task add:helm:repos
-          task update:helm:deps
+          task add:repos:helm
+          task update:deps:helm
           task lint
           task template
 
@@ -99,18 +99,18 @@ jobs:
         working-directory: applications/mlflow
         run: |
           # Update and package charts
-          task add:helm:repos
-          task update:helm:deps
-          task update:versions
-          task package
+          task add:repos:helm
+          task update:deps:helm
+          task update:versions:chart
+          task package:charts
 
       # Extract MLflow chart version for reference
       - name: Extract MLflow chart version
         id: chart-version
         working-directory: applications/mlflow
         run: |
           # Extract MLflow chart version using taskfile variable
-          CHART_VERSION=$(task -s version:extract)
+          CHART_VERSION=$(task -s extract:version:chart)
           echo "chart_version=$CHART_VERSION" >> $GITHUB_OUTPUT
           echo "Using MLflow chart version: $CHART_VERSION"
 
@@ -195,7 +195,7 @@ jobs:
           
           # Get license ID using the task - capture only the last line of output
           echo "Getting license ID..."
-          INSTALLATION_ID=$(CUSTOMER_NAME="$CUSTOMER_NAME" task customer:get-license-id | tail -n 1)
+          INSTALLATION_ID=$(CUSTOMER_NAME="$CUSTOMER_NAME" task get:license-id:customer | tail -n 1)
           
           # Check if we got a result
           if [ -z "$INSTALLATION_ID" ]; then
@@ -241,7 +241,7 @@ jobs:
           echo "$KUBECONFIG" > "$KUBECONFIG_FILE"
           
           # Run task to test Helm installation
-          KUBECONFIG="$KUBECONFIG_FILE" MLFLOW_VALUES="${{ matrix.config.values_file }}" task test:helm
+          KUBECONFIG="$KUBECONFIG_FILE" MLFLOW_VALUES="${{ matrix.config.values_file }}" task test:install:helm
         env:
           KUBECONFIG: ${{ steps.create-cluster.outputs.cluster-kubeconfig }}
           REPLICATED_APP: ${{ env.APP_SLUG }}
@@ -255,7 +255,7 @@ jobs:
         working-directory: applications/mlflow
         run: |
           # Run task to test application
-          task test:run-app-tests
+          task run:tests:app
         env:
           KUBECONFIG: ${{ steps.create-cluster.outputs.cluster-kubeconfig }}
           PORT: 5000
@@ -349,7 +349,7 @@ jobs:
           # Try to download license
           echo "Attempting to download license..."
           set +e
-          CUSTOMER_NAME="$CUSTOMER_NAME" task customer:download-license
+          CUSTOMER_NAME="$CUSTOMER_NAME" task download:license:customer
           DOWNLOAD_RESULT=$?
           set -e
           
@@ -515,7 +515,7 @@ jobs:
         working-directory: applications/mlflow
         run: |
           # Run task to test application
-          task test:run-app-tests
+          task run:tests:app
         env:
           KUBECONFIG: ${{ steps.create-cluster.outputs.cluster-kubeconfig }}
           PORT: 5000

applications/mlflow/README.md

@@ -88,40 +88,43 @@ The MLflow application includes a Taskfile.yml that allows running the same test
 - Existing Kubernetes cluster configured in your current context
 - kubectl, helm, and python3 installed
 
-### Common Testing Tasks
+### Common Development Tasks
+
+Tasks in our Taskfile follow a `verb:resource[:subresource]` naming convention that clearly describes their purpose.
 
 ```bash
 # Show available tasks
 task
 
-# Lint charts
-task lint
-
-# Template charts (render to stdout)
-task template
-
-# Package charts
-task package
+# Lint and validate charts
+task lint                 # Lint Helm charts
+task template             # Render templates to stdout (SDK disabled)
+task check:versions       # Verify Chart.yaml and KOTS manifest versions match
 
-# Run a local test with local charts (no Replicated registry)
-task test:local
+# Repository and dependency management
+task add:repos:helm       # Add required Helm repositories
+task update:deps:helm     # Update Helm chart dependencies
 
-# Run tests with charts from Replicated registry
-# Requires: REPLICATED_LICENSE_ID, REPLICATED_APP, REPLICATED_CHANNEL env vars
-task test:helm
+# Packaging and versioning
+task update:versions:chart # Update chart version refs in KOTS manifests
+task package:charts       # Package Helm charts for distribution
+task extract:version:chart # Extract current MLflow chart version
 
-# Run application tests against a running MLflow instance
-task test:run-app-tests
+# Installation testing
+task test:install:helm    # Install charts from Replicated registry
+task test:install:local   # Install charts from local directory (no registry)
+task test:install:kots    # Install application via KOTS
 
-# Extract MLflow chart version from KOTS manifest
-task version:extract
+# Application testing
+task run:tests:app        # Run application tests against running MLflow
+task run:tests:all        # Run all tests (Helm install + app tests)
 
-# Create a release in Replicated (after packaging charts)
-# Optional env vars: REPLICATED_APP, CHANNEL
-task release:create
+# Release management
+task create:release       # Create a Replicated release
 
-# Clean up generated files
-task clean
+# Cleanup
+task clean:files:charts   # Clean packaged chart files
+task clean:all            # Clean all generated files
 ```
 
 ### Testing Charts from Replicated Registry
@@ -130,16 +133,24 @@ To test charts from the Replicated registry:
 
 1. Ensure you have a running Kubernetes cluster and your kubectl context is set correctly
 
-2. Set required environment variables:
+2. Login to the registry and set up required environment variables:
    ```bash
    export REPLICATED_LICENSE_ID=your_license_id
    export REPLICATED_APP=app_slug
    export REPLICATED_CHANNEL=channel_name
+   
+   # Login to the registry
+   task login:registry
    ```
 
-3. Run the Helm test:
+3. Run the Helm installation test:
    ```bash
-   task test:helm
+   task test:install:helm
+   ```
+
+4. Run application tests against the deployed instance:
+   ```bash
+   task run:tests:app
    ```
 
 ### Local Development Testing
@@ -148,13 +159,18 @@ For local development without using the Replicated registry:
 
 1. Ensure you have a running Kubernetes cluster and your kubectl context is set correctly
 
-2. Run the local test:
+2. Run the local installation test:
    ```bash
    # Run test with local charts
-   task test:local
+   task test:install:local
 
    # Optionally specify a custom values file
-   MLFLOW_VALUES=./my-values.yaml task test:local
+   MLFLOW_VALUES=./my-values.yaml task test:install:local
+   ```
+
+3. Run application tests against the deployed instance:
+   ```bash
+   task run:tests:app
    ```
 
 This allows rapid iteration on chart changes without needing to publish to the Replicated registry.
@@ -165,16 +181,16 @@ The Taskfile provides tasks for packaging charts and creating releases in Replic
 
 ```bash
 # Package the charts (updates the version in the KOTS manifests)
-task package
+task package:charts
 
 # Extract the current chart version
-task version:extract
+task extract:version:chart
 
 # Create a release with the default app and channel
-task release:create
+task create:release
 
 # Create a release with custom app and channel
-REPLICATED_APP=myapp CHANNEL=beta task release:create
+REPLICATED_APP=myapp REPLICATED_CHANNEL=beta task create:release
 ```
 
 This enables you to package and publish releases directly from your local environment.