Skip to content

Allow custom boot/connect timeouts for testcloud VMs#2677

Merged
psss merged 1 commit intomainfrom
testcloud-control-boot-connect-timeouts
Feb 15, 2024
Merged

Allow custom boot/connect timeouts for testcloud VMs#2677
psss merged 1 commit intomainfrom
testcloud-control-boot-connect-timeouts

Conversation

@happz
Copy link
Copy Markdown
Contributor

@happz happz commented Feb 9, 2024

Pull Request Checklist

  • implement the feature
  • write the documentation

@happz happz added this to the 1.32 milestone Feb 9, 2024
@happz
Copy link
Copy Markdown
Contributor Author

happz commented Feb 9, 2024

/packit test

@lukaszachy lukaszachy force-pushed the testcloud-control-boot-connect-timeouts branch from 1d1e900 to 9f2e070 Compare February 9, 2024 14:34
@happz
Copy link
Copy Markdown
Contributor Author

happz commented Feb 9, 2024

/packit test

@happz happz force-pushed the testcloud-control-boot-connect-timeouts branch from 9f2e070 to dd43fba Compare February 12, 2024 12:08
@lukaszachy
Copy link
Copy Markdown
Contributor

Shouldn't we add __versionadded label?

@happz
Copy link
Copy Markdown
Contributor Author

happz commented Feb 12, 2024

Shouldn't we add __versionadded label?

Added in ebed80d

@happz
Copy link
Copy Markdown
Contributor Author

happz commented Feb 13, 2024

/packit test

1 similar comment
@happz
Copy link
Copy Markdown
Contributor Author

happz commented Feb 13, 2024

/packit test

@happz
Copy link
Copy Markdown
Contributor Author

happz commented Feb 13, 2024

I'm looking for inputs from @frantisekz, the failure of parallel test stinks with race conditions :/

Copy link
Copy Markdown
Member

@psss psss left a comment

Choose a reason for hiding this comment

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

Looks good, just two suggestions.

@happz happz force-pushed the testcloud-control-boot-connect-timeouts branch from 01b9a8a to 00b8b8d Compare February 15, 2024 08:31
Copy link
Copy Markdown
Member

@psss psss left a comment

Choose a reason for hiding this comment

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

Looks good, just two questions.

@psss psss added step | provision Stuff related to the provision step plugin | testcloud The testcloud virtual provision plugin labels Feb 15, 2024
Copy link
Copy Markdown
Member

@psss psss left a comment

Choose a reason for hiding this comment

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

Thanks for this improvement!

@psss psss self-assigned this Feb 15, 2024
@psss psss added the ci | full test Pull request is ready for the full test execution label Feb 15, 2024
@psss psss force-pushed the testcloud-control-boot-connect-timeouts branch from 297b643 to 846e2a7 Compare February 15, 2024 11:38
@psss
Copy link
Copy Markdown
Member

psss commented Feb 15, 2024

I'm looking for inputs from @frantisekz, the failure of parallel test stinks with race conditions :/

Yeah, looks like that. Filed #2687 to track this. Everything else passed, let's not block the PR on this.

@psss psss merged commit 846e2a7 into main Feb 15, 2024
@psss psss deleted the testcloud-control-boot-connect-timeouts branch February 15, 2024 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci | full test Pull request is ready for the full test execution plugin | testcloud The testcloud virtual provision plugin step | provision Stuff related to the provision step

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants