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

Print Hive installer logs if failure occurs during e2e #4008

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Print Hive installer logs if failure occurs during e2e #4008

wants to merge 2 commits into from

Conversation

kimorris27
Copy link
Contributor

Which issue this PR addresses:

Fixes

What this PR does / why we need it:

Test plan for issue:

Is there any documentation that needs to be updated for this PR?

How do you know this will function as expected in production?

@kimorris27
Copy link
Contributor Author

/azp run ci

@kimorris27 kimorris27 self-assigned this Dec 12, 2024
@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kimorris27 kimorris27 added the chainsaw Pull requests or issues owned by Team Chainsaw label Dec 12, 2024
@kimorris27 kimorris27 force-pushed the kimorris27/hotfix-e2e-hive-install-timeout branch from 1136bf3 to 21ecd3c Compare December 12, 2024 18:28
@kimorris27
Copy link
Contributor Author

/azp run ci

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kimorris27 kimorris27 force-pushed the kimorris27/hotfix-e2e-hive-install-timeout branch from 21ecd3c to c7c9802 Compare December 12, 2024 22:19
@kimorris27
Copy link
Contributor Author

/azp run ci

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

edisonLcardenas
edisonLcardenas requested changes Dec 13, 2024
View reviewed changes
Copy link
Collaborator

@edisonLcardenas edisonLcardenas left a comment

Choose a reason for hiding this comment

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

"ST1005: error strings should not be capitalized (stylecheck)" (I honestly hate this check.)

@kimorris27
Copy link
Contributor Author

/azp run ci

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kimorris27 kimorris27 force-pushed the kimorris27/hotfix-e2e-hive-install-timeout branch from c7c9802 to fbe3a13 Compare December 13, 2024 16:43
@kimorris27
Copy link
Contributor Author

/azp run ci

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kimorris27 kimorris27 force-pushed the kimorris27/hotfix-e2e-hive-install-timeout branch from fbe3a13 to 4ac4b2c Compare December 13, 2024 16:59
@kimorris27
Copy link
Contributor Author

/azp run ci

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kimorris27 kimorris27 force-pushed the kimorris27/hotfix-e2e-hive-install-timeout branch from 4ac4b2c to 0fc9977 Compare December 13, 2024 17:18
@kimorris27
Copy link
Contributor Author

/azp run ci

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kimorris27 kimorris27 force-pushed the kimorris27/hotfix-e2e-hive-install-timeout branch 6 times, most recently from aa55e38 to 53bc5b1 Compare December 16, 2024 17:15
@kimorris27 kimorris27 force-pushed the kimorris27/hotfix-e2e-hive-install-timeout branch from 53bc5b1 to 7dad096 Compare December 17, 2024 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@edisonLcardenas edisonLcardenas edisonLcardenas requested changes

@jewzaam jewzaam Awaiting requested review from jewzaam jewzaam will be requested when the pull request is marked ready for review jewzaam is a code owner

@bennerv bennerv Awaiting requested review from bennerv bennerv will be requested when the pull request is marked ready for review bennerv is a code owner

@hawkowl hawkowl Awaiting requested review from hawkowl hawkowl will be requested when the pull request is marked ready for review hawkowl is a code owner

@rogbas rogbas Awaiting requested review from rogbas rogbas will be requested when the pull request is marked ready for review rogbas is a code owner

@petrkotas petrkotas Awaiting requested review from petrkotas petrkotas will be requested when the pull request is marked ready for review petrkotas is a code owner

@jharrington22 jharrington22 Awaiting requested review from jharrington22 jharrington22 will be requested when the pull request is marked ready for review jharrington22 is a code owner

@cblecker cblecker Awaiting requested review from cblecker cblecker will be requested when the pull request is marked ready for review cblecker is a code owner

@cadenmarchese cadenmarchese Awaiting requested review from cadenmarchese cadenmarchese will be requested when the pull request is marked ready for review cadenmarchese is a code owner

@UlrichSchlueter UlrichSchlueter Awaiting requested review from UlrichSchlueter UlrichSchlueter will be requested when the pull request is marked ready for review UlrichSchlueter is a code owner

@SudoBrendan SudoBrendan Awaiting requested review from SudoBrendan SudoBrendan will be requested when the pull request is marked ready for review SudoBrendan is a code owner

@yjst2012 yjst2012 Awaiting requested review from yjst2012 yjst2012 will be requested when the pull request is marked ready for review yjst2012 is a code owner

@jaitaiwan jaitaiwan Awaiting requested review from jaitaiwan jaitaiwan will be requested when the pull request is marked ready for review jaitaiwan is a code owner

@anshulvermapatel anshulvermapatel Awaiting requested review from anshulvermapatel anshulvermapatel will be requested when the pull request is marked ready for review anshulvermapatel is a code owner

@hlipsig hlipsig Awaiting requested review from hlipsig hlipsig will be requested when the pull request is marked ready for review hlipsig is a code owner

@tiguelu tiguelu Awaiting requested review from tiguelu tiguelu will be requested when the pull request is marked ready for review tiguelu is a code owner

@mociarain mociarain Awaiting requested review from mociarain mociarain will be requested when the pull request is marked ready for review mociarain is a code owner

@tsatam tsatam Awaiting requested review from tsatam tsatam will be requested when the pull request is marked ready for review tsatam is a code owner

@bitoku bitoku Awaiting requested review from bitoku bitoku will be requested when the pull request is marked ready for review bitoku is a code owner

@fahlmant fahlmant Awaiting requested review from fahlmant fahlmant will be requested when the pull request is marked ready for review fahlmant is a code owner

Requested changes must be addressed to merge this pull request.

Assignees

@kimorris27 kimorris27

Labels
chainsaw Pull requests or issues owned by Team Chainsaw
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kimorris27 @edisonLcardenas