Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@caviri
Copy link
Member

@caviri caviri commented Jul 3, 2024

v0.4.0 Release

sabinem and others added 30 commits June 24, 2024 11:30
project forlder name is derived from the execution title
on folder create there is always a new folder created in the
users workspace so that the user will be able to easily run the
selected execution
clone the component repo in such a way that the submodule is taken
from the component repo commit: this needs to first check out
the component commit and the fetch the submodule
clone the component repo in such a way that the submodule is taken from
the component repo commit: this needs to first check out the component
commit and the fetch the submodule
project forlder name is derived from the execution title on folder
create there is always a new folder created in the users workspace so
that the user will be able to easily run the selected execution
* fix(compose.prod.yml): removing app volume

* typo(JSON_EDITOR): Correcting typo in variables

* chore(compose.yml): Proposing compose.yml to avoid change in documentation with -f.
the variable had the wrong value
This PR fixes an env variable setting in the compose files
#157)

feat(logs & output collections): Added missing collections in initial configuration
* fix(S3 Initiation): Fix for S3 initial setup

* fix(exception): Exception applied on CLI and logging.

* chore(cli/s3.py): adapt method to return value change

* Erasing assignation.

---------

Co-authored-by: sabinem <[email protected]>
#155)

fix(dependencies): Removing pyarrow and freezing numpy to <2 to avoid errors and warnings
that way the log will be visible right away in the file system
if if the docker run command fails at some point
this seems to be essential for long running container and also
in cases of ephemeral containers when an error occurs
simplify the output and log streaming command: both just need the
project path
simplify the workflow and disable buttons after selection or when
proceesing is not ready

submit run and then offer a button to view log files
logs can only be offered in the gui and cli per step
os.system does not throw an error, but keyboard interruptions
make sense here
* feat(execution-timestamps)

* feat(excecution & step): Added Creation/Update timestamp. Removed start/ending from creation.

* feat(step timestamps logs): Added timestamps on orchestrator.
Change `docker run` command and add the logs directory as an extra
volume, so that the logs will be visible right away and are also there
for persistent component or in error runs of ephemeral compoents
This PR
- depends on #175 to have
accessible logs

CLI Features:
- adds cli command `odtp logs` with argument `--project-path` to watch
the logs
- simplifies command `odtp output` to only have `--project-path` as an
argument

GUI Festures:
- simplify run page: disable and enable buttons depending on state
- submit execution and the provide access to logs
* feat(docker): Avoid rebuilding images available

* fix(attribute_name): self.container_names
Hello @sabinem, this is a hot-fix for a couple variables that I forgot
to rename. I tested it with `dt-example` and works properly now.
- delete unused functions
- log instead of print in the cli for some commands
* fix(dashboard): changed names of timestamp fields for execution

adapt to changes in the mongodb

* feat(dashboard executions): show start and end timestamps

in table view of executions show also start and end timestamps

---------

Co-authored-by: sabinem <[email protected]>
rename command for streaming logs
remove command for showing directory folder

Co-authored-by: sabinem <[email protected]>
* feat(logging): log commands to file to facilitate debugging

* feat(cli execution): add both print and logs

* chore(gitignore): add odtp.log to gitignore

* fix(run.py): don't log secrets to file

---------

Co-authored-by: sabinem <[email protected]>
sabinem and others added 4 commits July 3, 2024 11:51
fix(dashboard run page): fix minor styling issue

Co-authored-by: sabinem <[email protected]>
* feat(delete): CLI method to delete execution s3 and project folder

* feat(delete): CLI method to delete all MDB entries related to an execution

* feat(execution delete): Keep folder

* fix(project-path): Optional

* fix(execution): Remove typo and non used method

* fix(delete outputs): Fixed bug and added proejct_path as optional.

* Using print statements.

* chore(delete log): Logging only in debug

* fix(DT Entry): Wrong email field was retrieved.
@caviri caviri requested a review from sabinem July 3, 2024 12:03
@caviri caviri merged commit ca50f05 into main Jul 3, 2024
sabinem added a commit that referenced this pull request Jul 3, 2024
Co-authored-by: Carlos Vivar Rios <[email protected]>
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.

3 participants