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.

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

AZP/RELEASE: Fix launch condition-v1.15.x #9329

Merged
merged 1 commit into from
Sep 12, 2023

Conversation

Alexey-Rivkin
Copy link
Collaborator

@Alexey-Rivkin Alexey-Rivkin commented Aug 30, 2023

Cherry-picked from #9327.

@@ -139,6 +140,16 @@ function az_module_unload() {
module unload "${module}" || true
}

# Ensure that GPU is present
check_gpu() {
Copy link
Contributor

Choose a reason for hiding this comment

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

how come we back port only function definition and not back-porting any usage of this function?
either it's already defined, or not needed

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I stand corrected.
It's better to not add the GPU check in this PR.

yosefe
yosefe previously approved these changes Aug 30, 2023
Copy link
Contributor

@yosefe yosefe left a comment

Choose a reason for hiding this comment

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

pls squash

yosefe
yosefe previously approved these changes Aug 30, 2023
yosefe
yosefe previously approved these changes Aug 31, 2023
Copy link
Contributor

@yosefe yosefe left a comment

Choose a reason for hiding this comment

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

pls squash

@Alexey-Rivkin
Copy link
Collaborator Author

squashed

@yosefe
Copy link
Contributor

yosefe commented Sep 1, 2023

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s), but failed to run 1 pipeline(s).

@Alexey-Rivkin
Copy link
Collaborator Author

io_demo tests fail, as this PR doesn't include the io_demo setters fix #9332

artemry-nv
artemry-nv previously approved these changes Sep 1, 2023
@Alexey-Rivkin Alexey-Rivkin dismissed stale reviews from artemry-nv and yosefe via 65d4329 September 3, 2023 07:02
@Alexey-Rivkin Alexey-Rivkin force-pushed the topic/release_launch-v1.15.x branch 2 times, most recently from 5f16f78 to 5109d90 Compare September 3, 2023 07:17
@Alexey-Rivkin Alexey-Rivkin marked this pull request as draft September 4, 2023 05:05
auto-merge was automatically disabled September 4, 2023 05:05

Pull request was converted to draft

@yosefe
Copy link
Contributor

yosefe commented Sep 4, 2023

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s), but failed to run 1 pipeline(s).

@Alexey-Rivkin Alexey-Rivkin marked this pull request as ready for review September 11, 2023 07:36
@yosefe yosefe merged commit b629c15 into openucx:v1.15.x Sep 12, 2023
126 checks passed
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.

3 participants