kalleeh/gurum-cli

gurum-cli's Issues

Fix remaining todo's for gurum up around error handling.

gurum-cli/gurumcli/commands/up/__init__.py

Line 47 in 777362b

# TODO: We need to look at handling errors when there is no ~/Library/Application Support/gurum/.gurum file

gurum-cli/gurumcli/commands/up/__init__.py

Line 78 in 777362b

#TODO: Make this a helper.

gurum-cli/gurumcommon/gurum_manifest.py

Line 72 in 777362b

print(ex) #TODO raise UnableToReadManifestException()

Fix Login first time - configuration set up.

Login first run does not find the configuration file and creates a dummy file. Running login again exits with an error that the configuration file is invalid. Consider moving configuration wizard from login command to automatically start if configuration file is not found.

gurum init

Acceptance Criteria:

• Check that the file exists (gurum.yaml)
  If file doesn't exist, create generate a skeleton gurum.yaml.
• Validate gurum.yaml against schema. (nevermind the edit - someone forgot to check-in the schema + skeleton)

Add CodeCommit support to gurum init to simplify app setup

gureume init

• Create CodeCommit repository
• Configure the remote & git credentials
• Initialize git repo or re-use existing

Corresponding flow from CLI should be.
git push gureume master

Add context awareness to CLI and re-structure "advanced" options

Make CLI inherit application context from local active folder, add advanced structure to interact more natively with API structure.

gurum up

Acceptance Criteria:

• Verify that a gurum.yaml exists in the directory where the command is being run.
• Validate that it complies with schema.
• Prompt for PAT token to GitHub to setup CodePipeline, if it's GitHub.
  • Validate the PAT token against GitHub.
• Loop through the gurum.yaml and orchestrate creation,
  • apps
  • pipelines
  • services

Rename plural commands to singular

When running commands we use the plural variations gureume apps.
It might be best to change these to singular e.g. gureume app as we will be interacting with a single app.

Remove create, destroy and update commands from CLI

gurum up is preferred method now so should clean up create and update commands to defer users from using that.

Login - Region Overwritten

If your primary region is not eu-west-1 then it will be overwritten when running the login command.

We will need to ensure the region chosen during setup is used instead.

Setup CI pipeline

Running tests on PR, block if failing.
Automatic versioning.
Create GitHub release when merged to master

Enh: Split configuration file into two, for config and one for tokens

To simplify distribution of the CLI for a particular platform deployment it would be beneficial to be able to have the user pool id, client id, region etc as part of the git repository.
In this way new users could just git clone the CLI project and the log in to the platform with the correct settings.

Admin would need to modify this file before distributing in this organization to the correct cognito and region information.

Access tokens file should still stay in the same path as now, ~/Library/Application Support/gureume/.gureume

storage/handling of GitHub PAT in CLI

• Don't ask for token if user already specified it.
• Renew token if it has expired.
• gurum.yaml skeleton should point to a public repo to enable direct deployment if desired

Add support for service.yaml file

Add support for service.yaml file, example below

service:
  name: my-service
environments:
  - dev
  - test
  - uat
  - approval
  - prod
backing-services:
  dynamodb:
    HashAttributeName:
    HashAttributeType:
    RangeAttributeName:
    RangeAttributeType:
    ReadCapacityUnits:
    WriteCapacityUnits:

Helper script - Cognito user missing email attribute

Using the helper script causes an application deployment error since email is not set, causing the gurum-owner tag to be empty. Update the script to ask for email information. Consider adding the cognito role policy to the api CloudFormation template and create the cognito group attached to the role policy through the helper script.

gureume login - error when trying to login to wrong account

If you run gureume login when you are under a different role on your machine you receive face the following error (not obvious what happened):

gureume login
User: sam
Password:
Logging in sam...
2019-08-05 21:53:31 Found credentials in shared credentials file: ~/.aws/credentials
Getting temporary STS credentials...
An error occurred (ResourceNotFoundException) when calling the GetId operation: IdentityPool 'us-east-1:833ebb77-57f5-4281-aa04-98a7a2996973' not found.
Parameter validation failed:
Invalid length for parameter IdentityId, value: 0, valid range: 1-inf
Traceback (most recent call last):
  File "/Users/samhendr/.virtualenvs/gureume-cli-ZvMZbjE2/bin/gureume", line 11, in <module>
    load_entry_point('gureume-cli', 'console_scripts', 'gureume')()
  File "/Users/samhendr/.virtualenvs/gureume-cli-ZvMZbjE2/lib/python3.7/site-packages/click/core.py", line 764, in __call__
    return self.main(*args, **kwargs)
  File "/Users/samhendr/.virtualenvs/gureume-cli-ZvMZbjE2/lib/python3.7/site-packages/click/core.py", line 717, in main
    rv = self.invoke(ctx)
  File "/Users/samhendr/.virtualenvs/gureume-cli-ZvMZbjE2/lib/python3.7/site-packages/click/core.py", line 1137, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/samhendr/.virtualenvs/gureume-cli-ZvMZbjE2/lib/python3.7/site-packages/click/core.py", line 956, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/samhendr/.virtualenvs/gureume-cli-ZvMZbjE2/lib/python3.7/site-packages/click/core.py", line 555, in invoke
    return callback(*args, **kwargs)
  File "/Users/samhendr/.virtualenvs/gureume-cli-ZvMZbjE2/lib/python3.7/site-packages/click/decorators.py", line 64, in new_func
    return ctx.invoke(f, obj, *args, **kwargs)
  File "/Users/samhendr/.virtualenvs/gureume-cli-ZvMZbjE2/lib/python3.7/site-packages/click/core.py", line 555, in invoke
    return callback(*args, **kwargs)
  File "/Users/samhendr/repositories/gureume/gureume-cli/gureumecli/commands/login/__init__.py", line 52, in cli
    do_cli(ctx, user, password)  # pragma: no cover
  File "/Users/samhendr/repositories/gureume/gureume-cli/gureumecli/commands/login/__init__.py", line 170, in do_cli
    ctx._config.set('default', 'aws_access_key_id', credentials['aws_access_key_id'])
KeyError: 'aws_access_key_id'

We need to look into this as we either shouldn't be getting this error or we need to clean it up.

gurum down

Acceptance Criteria:

• Verify that a gurum.yaml exists in the directory where the command is being run.
• Validate that it complies with schema.
• Loop through the gurum.yaml and orchestrate deletion of apps, pipelines, services

Deletion of App throws a 500

This usually takes a couple of minutes...
This call is asynchrounous so feel free to Ctrl+C anytime and it will continue running in background.
|[500] Server Error
Unknown Error:```