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

31 bug fix workspace cli command #40

Merged
merged 3 commits into from
May 28, 2024
Merged

Conversation

odarotto
Copy link
Collaborator

No description provided.

@odarotto odarotto requested a review from shinybrar May 27, 2024 22:04
@odarotto odarotto linked an issue May 27, 2024 that may be closed by this pull request
4 tasks
Copy link
Contributor

@shinybrar shinybrar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall great PR, I will fix the few small things and merge it.

workflow/cli/workspace.py Outdated Show resolved Hide resolved
@shinybrar shinybrar merged commit b0a7c8c into main May 28, 2024
2 checks passed
shinybrar added a commit that referenced this pull request Jun 4, 2024
* refactor(cli/schedules.py): separating schedules functions from pipelines

Schedules client created to handle all operations related to schedules

* feat(cli): improvements to pipelines and schedules commands

* refactor(schedules.py): adding section when using details option

* feat(variables.py): adding missing statuses

* feat(cli): adding configs commands

confgis commands added so the cli can work with the new version of pipelines(^2.7.0)

* refactor(http/configs.py): removing print statement

* feat(cli/schedules.py): fixing bad formatting

* fix(cli): addressing comments on PR

* feat(docker-compose-tutorial.yml): adding docker compose for tutorial (#35)

* 32 feat create tutorial workspace (#33)

* feat(workspaces/tutorial.yml): adding tutorial workspace

* feat(workspaces/tutorial.yml): adding baseurl for configs

* feat(cli): adding configs commands

confgis commands added so the cli can work with the new version of pipelines(^2.7.0)

* refactor(http/configs.py): removing print statement

* feat(cli/schedules.py): fixing bad formatting

* fix(cli): addressing comments on PR

* test(test_http_context.py): adding tests

Tests added for pipelines client on HTTPContext.

* ci(ci.yml): adding steps to tests job

Adding steps for: starting docker services needed for tests; setting development workspace and killing services after tests are completed.

* ci(ci.yml): changing docker-compose file name

* ci(ci.yml): adding docker login step

* feat(cli/main.py): adding logging on top level function (#37)

Logging info about actual workspace being used.

* 31 bug fix workspace cli command (#40)

* fix(cli/workspace.py): changing filename for active workspace

* feat(workflow/http/context.py): adding validator function for workspace field

* fix(workspace): fixed workspace cli commands and suppressed UserWarning from pydantic_settings

---------

Co-authored-by: Shiny Brar  (he/il) <[email protected]>

* chore(main): release 0.4.0 (#26)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* feat(cli): adding configs commands

confgis commands added so the cli can work with the new version of pipelines(^2.7.0)

* refactor(http/configs.py): removing print statement

* feat(cli/schedules.py): fixing bad formatting

* fix(cli): addressing comments on PR

* feat(cli): improvements to pipelines and schedules commands

* fix(cli): addressing comments on PR

* test(test_http_context.py): adding tests

Tests added for pipelines client on HTTPContext.

* ci(ci.yml): adding steps to tests job

Adding steps for: starting docker services needed for tests; setting development workspace and killing services after tests are completed.

* ci(ci.yml): changing docker-compose file name

* ci(ci.yml): adding docker login step

* fix(workflow/utils/read.py): changing active workspace filename

* fix(cli/schedules.py): fixing variable name

* feat(workflow-dev): added workspace configuration for live system tests

* fix(http): fixed pipelines,schedules and configs to use baseurls with prefixing versioning

* fix(workspace-configs): updated workspace configs to point to v2

* refactor(test_http_context.py): fixing tests

* refactor(workspaces/development.yml): adding version suffix to baseurls

* Retrying

* fix(docker-compose-tutorial.yml): settings services to latests version

* fix(docker-compose-tutorial.yml): adding local service

---------

Co-authored-by: Shiny Brar  (he/il) <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] Fix workspace CLI Command
2 participants