test(cd script update): add improved e2e cd script #4192
Conversation
github-actions
bot
added
the
remind-reviewers
PranjalC100
requested review from
meet2mky
and removed request for
raj-prince
| GCS_DEST="gs://${BUCKET_NAME_TO_USE}/v${VERSION}/${COMMIT_HASH}/${VM_INSTANCE_NAME}/" | ||
|
|
||
| # Upload the consolidated log with fixed name for pipeline compatibility | ||
| gcloud storage cp "$LOG_FILE" "${GCS_DEST}_combined_e2e_logs_${TIMESTAMP}.txt" |
There was a problem hiding this comment.
In case of failure directly print this file name ${GCS_DEST}_combined_e2e_logs_${TIMESTAMP}.txt so that we know exact file name
|
Hi @Tulsishah, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you! |
3 similar comments
|
Hi @Tulsishah, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you! |
|
Hi @Tulsishah, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you! |
|
Hi @Tulsishah, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you! |
PranjalC100
force-pushed
the
cpranjal-improved-e2e-cd-script
branch
from
77f59d5 to
e65b09a
Compare
|
Hi @Tulsishah, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you! |
1 similar comment
|
Hi @Tulsishah, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you! |
PranjalC100
force-pushed
the
cpranjal-improved-e2e-cd-script
branch
from
8692f22 to
5143da4
Compare
|
Hi @Tulsishah, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you! |
PranjalC100
force-pushed
the
cpranjal-improved-e2e-cd-script
branch
5 times, most recently
from
d7cf5f8 to
62342b1
Compare
|
Hi @Tulsishah, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you! |
1 similar comment
|
Hi @Tulsishah, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you! |
PranjalC100
force-pushed
the
cpranjal-improved-e2e-cd-script
branch
2 times, most recently
from
830f2c7 to
cc6de93
Compare
Local improved e2e cd script, a better cleanup is req faster run tial, commit to be reverted testing with updated rhel conditions Add header guard to prevent multiple sourcing Rebase and add print statement for logs
PranjalC100
force-pushed
the
cpranjal-improved-e2e-cd-script
branch
from
cc6de93 to
42e6c63
Compare
|
Hi @Tulsishah, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you! |
2 participants
Description
Link to the issue in case of a bug fix.
Testing details
Any backward incompatible change? If so, please explain.