Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(pipelines): Add environments #220

Merged
merged 5 commits into from
Nov 29, 2021
Merged

feat(pipelines): Add environments #220

merged 5 commits into from
Nov 29, 2021

Conversation

raulb
Copy link
Member

@raulb raulb commented Nov 23, 2021

Description of change

Depends on https://github.com/meroxa/meroxa-go/pull/84

Type of change

  • New feature
  • Bug fix
  • Refactor
  • Documentation

How was this tested?

  • Unit Tests
  • Tested in staging

Demo

Before this pull-request

❯ meroxa pipelines ls                              
  ID               NAME                STATE  
====== ============================= =========
  762   meroxa-cli-to-slack/discord   healthy 
   11             default             healthy 
 1910            with-env             healthy

❯ .m pipelines describe default
Manage pipelines on Meroxa

Usage:
  meroxa pipelines [command]

Aliases:
  pipelines, pipeline

Available Commands:
  create      Create a pipeline
  list        List pipelines
  remove      Remove pipeline
  update      Update pipeline name, state or metadata
...

❯ meroxa pipelines create another-with-env --env 1a08cdec-64b2-40b1-9de4-a20477852de9
Error: unknown flag: --env

After this pull-request

❯ .m pipelines ls
                 UUID                    ID               NAME               ENVIRONMENT    STATE  
====================================== ====== ============================= ============= =========
 fd1052f6-b728-4996-9122-28bba4112363    762   meroxa-cli-to-slack/discord     common      healthy 
 5ccbadec-b3c4-430f-bd5c-942414e9890a     11             default               common      healthy 
 4ff99dae-470d-4b50-b937-fd6a36308b4b   1910            with-env                 foo       healthy 

❯ .m pipelines describe default
  UUID:   5ccbadec-b3c4-430f-bd5c-942414e9890a 
    ID:   11                                   
  Name:   default                              
 State:   Healthy  

❯ .m pipelines describe with-env
             UUID:   4ff99dae-470d-4b50-b937-fd6a36308b4b 
               ID:   1910                                 
             Name:   with-env                             
 Environment UUID:   1a08cdec-64b2-40b1-9de4-a20477852de9 
 Environment Name:   foo                                  
            State:   Healthy  

❯ .m pipelines create with-env --env foo
Creating pipeline "another-with-env" in "foo" environment...
Pipeline "with-env" successfully created!

❯ .m pipelines describe with-env               
             UUID:  3db69335-4c24-40f6-ba30-193cfa2b11a3
               ID:   1914                                 
             Name:   with-env                     
 Environment UUID:   1a08cdec-64b2-40b1-9de4-a20477852de9 
 Environment Name:   foo                                  
            State:   Healthy  

❯ .m pipelines create another-with-env --env 1a08cdec-64b2-40b1-9de4-a20477852de9
Creating pipeline "another-with-env" in "1a08cdec-64b2-40b1-9de4-a20477852de9" environment...
Pipeline "another-with-env" successfully created!

❯ .m pipelines describe another-with-env               
             UUID:   985ee31a-b8d2-4990-a0fd-cff7a79ea0bc 
               ID:   1915                                 
             Name:   another-with-env                     
 Environment UUID:   1a08cdec-64b2-40b1-9de4-a20477852de9 
 Environment Name:   foo                                  
            State:   Healthy  

Additional references

It also makes some other changes that are timely as we use more UUIDs everywhere.

@raulb raulb force-pushed the raul/env-pipelines branch 4 times, most recently from 7daf6b9 to 238cb8e Compare November 23, 2021 15:21
@raulb raulb requested a review from janelletavares November 23, 2021 15:45
@raulb raulb self-assigned this Nov 23, 2021
@raulb raulb marked this pull request as ready for review November 23, 2021 15:45
@raulb raulb force-pushed the raul/env-pipelines branch from 94a92fa to a14406e Compare November 29, 2021 12:07
@raulb raulb merged commit 8cebda9 into master Nov 29, 2021
@raulb raulb deleted the raul/env-pipelines branch November 29, 2021 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants