Skip to content

Rc/staging release 20240221 - #127

Merged
andricDu merged 24 commits into
mainfrom
rc/staging-release-20240221
Feb 22, 2024
Merged

Rc/staging release 20240221#127
andricDu merged 24 commits into
mainfrom
rc/staging-release-20240221

Conversation

@colorzzr

Copy link
Copy Markdown
Member

Summary

staging release 20240221:

  • optimize resumable upload logic:
    • update cli logic to allow resume an upload with different chuck size @hugoantunes
    • change the log name to resumable_upload_log.log @colorzzr
  • add new command file metadata to download metadata/tags/attributes of a specific file @colorzzr
  • add proper handler to invalid token error @colorzzr
  • update logic of configuration@colorzzr:
    • unify configuration for multi-tenancy
    • make config_path as env variable for nextflow pipeline
  • update cli logic to adopt project folder implementation @colorzzr
  • add new command file move to allow moving and renaming a file or folder @colorzzr

JIRA Issues

(What JIRA issues this merge request is related to)

Type of Change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
  • Refactor or reformatting

Testing

Are there any new or updated tests to validate the changes?

  • Yes
  • No

Test Directions

(Additional instructions for how to run tests or validate functionality if not covered by unit tests)

Hugo Seixas Antunes and others added 23 commits November 30, 2023 16:55
…#112)

- update GET /v1/files/chunks/presigned api to pass the chunk_size as parameter.
- set chunk_size to upload_chunk in stream_upload
* PILOT-3962: Port over changes from 2.7.3 to 2.7.4

* prepare the release branch

* use correct version

* add new command metadata to download file metadata/tags/attributes

* add new command for downloading file metadata

* add test cases for metadata download

* remove the testing data

* rename the output file name

* reformat the attribute file with the template name

* bumpup into next version

---------

Co-authored-by: Daniel <dfelipe@indocresearch.org>
Co-authored-by: zhiren <zzhan@indocresearch.org>
Co-authored-by: Dušan Andrić <Dusan.Andric@gmail.com>
…115)

* add new command metadata to download file metadata/tags/attributes

* update the upload command to read tag/attribute file when upload

* fixup test cases

* bumpup to next version

---------

Co-authored-by: zhiren <zzhan@indocresearch.org>
…gateway (#117)

* add the new error handler for 401 when portal has shared login session with cli

* add the test cases for token refresh

* update version to next patch

---------

Co-authored-by: zhiren <zzhan@indocresearch.org>
* remove the unnecessary item type in search function

* testing

* update logic for project folder

* add the test case for project folder in list/upload api

* update download logic for project folder

* add the more test case for file download

* use correct default config

* bumup version

---------

Co-authored-by: zhiren <zzhan@indocresearch.org>
PILOT-4319: update config file to allow user customize the api_url and keycloak_url
* add the basic logic of file move/renaming command

* add new command for file move

* add the test cases for file move

* add the response in try exception

* bumpup versions

---------

Co-authored-by: zhiren <zzhan@indocresearch.org>
Patch20240109: update copyright to 2024
* add the new name in git ignore

* add the new name in git ignore

* remove the resumable logs after successful uploading

* fix up test cases

* update the logic of removing output file into a dedicate function

---------

Co-authored-by: zhiren <zzhan@indocresearch.org>
…-existing parent folders (#124)

* add a new logic to allow cli to create non-exist folder when moving or renaming

* add a new boolean option to skip prompt confirmation in pipeline

* add the test cases for file move client

* add a test case to check the error handling of an aggregated function

* bumpup versions

---------

Co-authored-by: zhiren <zzhan@indocresearch.org>
@colorzzr
colorzzr requested a review from andricDu February 21, 2024 20:09
@colorzzr colorzzr self-assigned this Feb 21, 2024
@github-actions

github-actions Bot commented Feb 21, 2024

Copy link
Copy Markdown
Contributor

Coverage

Coverage Report
FileStmtsMissCoverMissing
__init__.py00100% 
pilotcli.py0310%5, 7–8, 10–14, 17–21, 23–28, 31–39, 42–44
commands
   __init__.py00100% 
   container_registry.py01959%16, 22–24, 31–33, 41–44, 50–53, 62–65
   dataset.py04844%19, 36–38, 41–48, 50–60, 78–80, 83–91, 93–103, 123, 127
   entry_point.py01085%39, 45, 89–91, 93–97
   file.py04183%40, 137, 146–148, 152, 157, 160–162, 179–180, 222, 307–314, 316, 318, 332–338, 340, 369, 371, 374, 420, 423, 440–442, 448–449
   project.py0197%17
   user.py0685%23, 40, 48, 56, 72–73
configs
   __init__.py00100% 
   app_config.py00100% 
   config.py00100% 
   user_config.py01784%61, 70, 114, 124, 131, 135, 139, 143, 147, 151, 155, 159, 163, 167, 171, 175, 179
models
   __init__.py00100% 
   enums.py00100% 
   service_meta_class.py0180%9
   singleton.py00100% 
   upload_form.py0625%30, 42–46
resources
   custom_error.py00100% 
   custom_help.py00100% 
services
   __init__.py00100% 
services/container_registry_manager
   container_registry_manager.py010817%19–20, 23–26, 29–33, 36–44, 48–60, 62–64, 68–82, 84–86, 90–99, 101–103, 107–110, 125–136, 140–143, 147–162, 164, 166–169
services/crypto
   __init__.py00100% 
   crypto.py01940%35, 43–47, 49, 59–61, 69–75, 77, 79
services/dataset_manager
   dataset_detail.py02370%42, 44, 61, 68–73, 75–77, 79–88, 90
   dataset_download.py01983%56, 72–73, 84, 96, 98–99, 101–102, 104, 106–108, 116–119, 153, 165
   dataset_list.py0878%37, 43–46, 48–50
   model.py00100% 
services/file_manager
   __init__.py00100% 
   file_list.py01775%29, 54, 56, 70–72, 80–83, 85, 91–96
   file_manifests.py09525%19–24, 39–43, 47–48, 51–59, 61, 65–68, 71–75, 77, 81–82, 85–87, 91–92, 95–102, 104, 107, 109–110, 112–113, 115–118, 123–129, 134–139, 142–148, 150–152, 155–161, 163–165, 167–170
   file_tag.py03931%23, 27–31, 33, 36–48, 50–52, 54–55, 59–61, 64–68, 70–73, 75–76
services/file_manager/file_download
   __init__.py00100% 
   download_client.py017423%44–45, 47–53, 56–57, 59–60, 69, 71–72, 76–79, 86, 91–94, 96, 98–106, 108, 110, 114–119, 121, 124–125, 128–130, 132–134, 136–137, 139–141, 145–153, 156–161, 165–174, 182–184, 186–193, 195–199, 204–214, 216–217, 220–221, 226–227, 231–236, 240–241, 244–246, 248–259, 263–266, 269–273, 275–277, 279–281, 283–284, 286, 290–292, 294–307, 309
   model.py0187%15
services/file_manager/file_metadata
   __init__.py00100% 
   file_metadata_client.py0395%97–99
services/file_manager/file_move
   __init__.py00100% 
   file_move_client.py0593%79–81, 83, 110
services/file_manager/file_upload
   __init__.py00100% 
   exception.py0175%10
   file_upload.py02884%35–42, 99–101, 113, 144–145, 149, 154, 191, 211–213, 254–255, 257–259, 333, 336, 340
   models.py0591%24, 44, 150–152
   upload_client.py07760%101–104, 207–209, 223–229, 231–235, 237–249, 251, 295, 298, 302, 304–306, 308–311, 315–319, 321, 325, 327, 329, 331, 350, 380–381, 386, 390–393, 397, 413, 415–417, 424, 429–430, 432, 434–435, 437–440, 442
   upload_validator.py02156%29–31, 34–39, 42–48, 51–52, 57, 59, 61
services/logger_services
   __init__.py00100% 
   log_functions.py00100% 
services/output_manager
   __init__.py00100% 
   error_handler.py00100% 
   help_page.py00100% 
   message_handler.py05770%23, 45, 50, 65–67, 72, 82, 87, 100, 105, 112, 117, 122, 126, 142, 177, 187, 196, 214, 224, 243, 265–276, 287–292, 294–300, 311, 315–316, 320–321, 323–324, 328, 332, 336
services/project_manager
   __init__.py00100% 
   project.py0878%38, 44–47, 49–51
services/user_authentication
   __init__.py00100% 
   decorator.py0293%27, 30
   token_manager.py0986%26, 41–42, 73, 76, 91, 96, 100, 105
   user_login_logout.py04266%30–31, 41, 122, 126–132, 134–136, 140–141, 145–152, 154–156, 160–161, 168–173, 175–177, 181–184
utils
   __init__.py00100% 
   aggregated.py04068%50, 62, 102, 115–117, 119–120, 127–128, 130, 134–137, 139–140, 154–161, 163–164, 168–175, 177–179, 194–195
TOTAL307998168% 

@andricDu
andricDu merged commit 268b2af into main Feb 22, 2024
@colorzzr
colorzzr deleted the rc/staging-release-20240221 branch March 4, 2024 21:32
@colorzzr
colorzzr restored the rc/staging-release-20240221 branch March 4, 2024 21:32
colorzzr pushed a commit that referenced this pull request May 23, 2024
…40221

Rc/staging release 20240221

Former-commit-id: 9c5bf4b
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.

2 participants