Skip to content

Test/CV2-6381: Check ci is failing #105

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
Merged
danielevalverde merged 55 commits into from
Aug 11, 2025
Merged

Test/CV2-6381: Check ci is failing #105

danielevalverde merged 55 commits into from
Aug 11, 2025

Conversation

@danielevalverde
Copy link
Contributor

@danielevalverde danielevalverde commented Aug 9, 2025 •
edited
Loading

Description:

  • Improved Submodule Checkout to avoid conflict when switching branches
  • Added git pull command to ensure the submodules are always synchronized with the latest changes from their remote repositories on CI step and on first build script
  • Updated Test File: Commented out the presto images and updated the test file to reflect this change.

Reference: CV2-6381

@danielevalverde
new branch
0d9ed92
@danielevalverde
run builds on all branches
3b31907
@danielevalverde
more debugging
cbb0c01
@danielevalverde
more debugging
e4594dc
@danielevalverde
debug presto branch
4ce7762
@danielevalverde
debug
b7d4049
@danielevalverde
update first-build script
9971cbf
@danielevalverde
more debug
3a7cc16
@danielevalverde
debug presto
459d503
@danielevalverde
try to fix problem with presto
e749b64
@danielevalverde
try to fix conflict and change the presto branch
870fb5a
@danielevalverde
try to simplify github checkout command
ddbc5ff
@danielevalverde
fix syntax
3e98573
@danielevalverde
fix syntax
24e8288
@danielevalverde
fix syntax
d8a8f57
@danielevalverde
try to fix preto issues
ecc13c5
@danielevalverde
try to fix preto issues
d8b81c9
@danielevalverde
try to fix preto issues
f803bea
@danielevalverde
try to fix queue_worker issues
c63c5ed
@danielevalverde
try to fix queue_worker issues
cadbf00
@danielevalverde
try to fix queue_worker issues
15906a5
@danielevalverde
try to fix queue_worker issues
478ba9b
@danielevalverde
debug
fb288b4
@danielevalverde
debug
9fecb80
@danielevalverde
debug
8976847
@danielevalverde
comment presto images
7013531
@danielevalverde
debug
ed81c4d
@danielevalverde
debug alegre
f612b86
@danielevalverde
debug Alegre
1141e79
@danielevalverde
test Alegre
5d6452e
@danielevalverde
debug check web error
2a28a20
@danielevalverde
debug only check web
a01c4ef
@danielevalverde
more check web debug
3926d23
@danielevalverde
testing...
4be55f3
@danielevalverde
test if it is a cache problem
721d213
@danielevalverde
commend pull check web
cb50598
@danielevalverde
try to fix: submodules checkout to fix conflicts
70a2e46
@danielevalverde
try to fix: submodules checkout to fix conflicts
2898f6b
@danielevalverde
try to fix: submodules checkout to fix conflicts
ae26ca9
@danielevalverde
try to fix: submodules checkout to fix conflicts
d6559ba
@danielevalverde
try to fix: submodules checkout to fix conflicts
dbc657b
@danielevalverde
try to fix: submodules checkout to fix conflicts
525627e
@danielevalverde
try to fix: submodules checkout to fix conflicts
cb26bc3
@danielevalverde
debug containers
d35578c
@danielevalverde
add 'git pull' comand
7d1566c
@danielevalverde
change branch order in checkout command
f38e27a
@danielevalverde
change branch order in checkout command 2
9ce4461
@danielevalverde
back with -d flag
209419c
@danielevalverde
update test after comment presto images
316f1db
@danielevalverde
clean up debugs
3d1acd8
@danielevalverde
clean up debugs
d1c23cd
@danielevalverde
clean up debugs
647b3c8
@danielevalverde
clean up debugs
a35b15d
@danielevalverde
run
9daac3f
@danielevalverde danielevalverde marked this pull request as ready for review August 9, 2025 20:11
@danielevalverde danielevalverde requested a review from caiosba August 9, 2025 20:11
caiosba
caiosba approved these changes Aug 10, 2025
View reviewed changes
Copy link
Contributor

@caiosba caiosba left a comment

Choose a reason for hiding this comment

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

Dani and I paired on a few things here, so it was an easy review... thanks for the perseverance, Dani! Great work!

@danielevalverde danielevalverde changed the title Test/cv2 6381 check ci is failing Test/CV2-6381: Check ci is failing Aug 11, 2025
@danielevalverde danielevalverde merged commit 3a076c0 into develop Aug 11, 2025
1 check passed
@danielevalverde danielevalverde deleted the test/CV2-6381-check-ci-is-failing branch August 11, 2025 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@caiosba caiosba caiosba approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@danielevalverde @caiosba