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.

Sign up for GitHub

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

Jump to bottom

BUG - Azure download fails for some analyses #385

Open
UmmulkiramR opened this issue Nov 22, 2023 · 1 comment · Fixed by #386
Open

BUG - Azure download fails for some analyses #385

UmmulkiramR opened this issue Nov 22, 2023 · 1 comment · Fixed by #386
Assignees
@UmmulkiramR
Labels
bug Something isn't working

Comments

@UmmulkiramR
Copy link
Contributor

UmmulkiramR commented Nov 22, 2023 •
edited
Loading

The following error was observed when running download for the below analyses:

image

Need to investigate and fix.
@UmmulkiramR UmmulkiramR added the bug Something isn't working label Nov 22, 2023
@UmmulkiramR UmmulkiramR self-assigned this Nov 22, 2023
@edsu7 edsu7 mentioned this issue Nov 22, 2023
Open
@UmmulkiramR UmmulkiramR changed the title BUG - BUG - Azure download fails for some analysis Nov 22, 2023
@UmmulkiramR UmmulkiramR changed the title BUG - Azure download fails for some analysis BUG - Azure download fails for some analyses Nov 22, 2023
UmmulkiramR pushed a commit that referenced this issue Nov 22, 2023
fix for issue #385
88d71d6
UmmulkiramR added a commit that referenced this issue Nov 22, 2023
@UmmulkiramR
fix for issue #385 (#386)
11337b5 
Co-authored-by: UmmulkiramR <urangwala@oicr.on.ca>
@UmmulkiramR UmmulkiramR linked a pull request Nov 24, 2023 that will close this issue
fix for issue #385 #386
Merged
UmmulkiramR pushed a commit that referenced this issue Nov 27, 2023
rc release. Includes
6b1a73d 
 - auto sync storage profiles - #387
 - azure download fails - #385
 - keycloak permission manager- #373
@UmmulkiramR UmmulkiramR mentioned this issue Nov 27, 2023
Merged
UmmulkiramR added a commit that referenced this issue Nov 27, 2023
@UmmulkiramR @MitchellShiell
@dahiyaAD @joneubank @leoraba
Score Release RC/5.10.0 (#388)
098cafe 
* Readme updated

* mergeback for 5.9.1-SNAPSHOT

* updated readme copy

* updated readme copy

* Update README.md

Co-authored-by: Jon Eubank <joneubank@gmail.com>

* CORS configurations for score server allowing multiple origins at once - #367

* Organizing score-server application.yml

* docker JRE ireplacing JRE alpine image to support multiple architecturesmage multi arch supported

* latest ubuntu LTS 22.04 score client

* using JDK image as builder

* update docker dind (#376)

from docker image docker:18.06-dind to docker:20.10-dind

* fix docker dind TLS (#379)

* increase Jenkins timeout (#380)

from 30 to 45 mins

* Fix/jenkins extend timeout (#381)

* increase Jenkins timeout

from 30 to 45 mins

* increase Jenkins timeout

from 45 to 60 mins

* increase jenkins timeout

* update DeployWithHelm job name in Jenkinsfile (#383)

* fix for issue #385 (#386)

Co-authored-by: UmmulkiramR <urangwala@oicr.on.ca>

* Auto sync storage profiles #378 (#382)

* added server endpoint to get profiles

* change to switch score-client storage implementations

* cleared azure and s3 related entries app.yml

* some bug fixes and enhancements

* updated comment

* added test profile.

* added test profile.

* refactored code based on review comments - BaseController now returns a single profile value. The actual profile name and the profile value returned by the api are now different.

* debug logging removed

* replaced profile value

* updated readme

* updated readme

* added a test profile

* review changes
- Storage profile values now come from an enum in score-core
- test configuration created to mock storage profile bean
- users will be able to provide a default profile value when working with old score-server instances

* users will be able to provide a default profile value when working with old score-server instances

* config change

* added a test config in score server

---------

Co-authored-by: UmmulkiramR <urangwala@oicr.on.ca>

* rc release. Includes
 - auto sync storage profiles - #387
 - azure download fails - #385
 - fix in Jenkins to build the Score docker image to run on amd/arm architectures and updating score-client image to use latest LTS Ubuntu- #374 

---------

Co-authored-by: Mitchell Shiell <59712867+MitchellShiell@users.noreply.github.com>
Co-authored-by: dahiyaAD <adahiya@oicr.on.ca>
Co-authored-by: dahiyaAD <123118704+dahiyaAD@users.noreply.github.com>
Co-authored-by: Jon Eubank <joneubank@gmail.com>
Co-authored-by: Leonardo Rivera <leorivera_88@hotmail.com>
Co-authored-by: UmmulkiramR <urangwala@oicr.on.ca>
justincorrigible added a commit that referenced this issue Nov 28, 2023
@justincorrigible
release 5.10.0 - bug fix for issue #385
f81f34a
@lindaxiang
Copy link

Thanks to @UmmulkiramR
I have tested the above all 3 analysis with score version 5.10.0, all were successfully downloaded.

[2c/fd152b] process > test_song_score_download_user:SONG_SCORE_DOWNLOAD:songGet (318e6eca-b1e3-4489-8e6e-cab1e34489e7) [100%] 1 of 1 ✔
[cc/26d027] process > test_song_score_download_user:SONG_SCORE_DOWNLOAD:scoreDn (318e6eca-b1e3-4489-8e6e-cab1e34489e7) [100%] 1 of 1 ✔

Completed at: 28-Nov-2023 17:32:48
Duration    : 20m 49s
CPU hours   : 0.2
Succeeded   : 2

[c5/545080] process > test_song_score_download_user:SONG_SCORE_DOWNLOAD:songGet (c80b8c18-d247-412e-8b8c-18d247112e9f) [100%] 1 of 1 ✔
[f0/dabd64] process > test_song_score_download_user:SONG_SCORE_DOWNLOAD:scoreDn (c80b8c18-d247-412e-8b8c-18d247112e9f) [100%] 1 of 1 ✔

Completed at: 28-Nov-2023 17:46:41
Duration    : 11m
CPU hours   : 0.2
Succeeded   : 2
  • 2ef0c59a-677b-5a85-98f6-ea1b7c37670c (analysisId: 6f8d30f8-70c9-45ec-8d30-f870c9f5ec08)
[33/ae6537] process > test_song_score_download_user:SONG_SCORE_DOWNLOAD:songGet (6f8d30f8-70c9-45ec-8d30-f870c9f5ec08) [100%] 1 of 1 ✔
[50/c38881] process > test_song_score_download_user:SONG_SCORE_DOWNLOAD:scoreDn (6f8d30f8-70c9-45ec-8d30-f870c9f5ec08) [100%] 1 of 1 ✔

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@UmmulkiramR UmmulkiramR

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix for issue #385 overture-stack/score
2 participants
@lindaxiang @UmmulkiramR