-
Notifications
You must be signed in to change notification settings - Fork 187
kola: Test interactive ISO login without networking #1330
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
kola: Test interactive ISO login without networking #1330
Conversation
82c88ce to
061f782
Compare
|
OK, tested and working in concert with coreos/fedora-coreos-config#339 ! |
| } | ||
|
|
||
| // SerialPipe reads the serial console output into a pipe | ||
| func (builder *QemuBuilder) SerialPipe() (*os.File, error) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is no longer used in this PR but I kept it because it'll surely be useful later.
061f782 to
544e488
Compare
544e488 to
024ae70
Compare
|
OK the only reason the test was passing here is I'd forgotten to add it to default scenario list. But now that coreos/fedora-coreos-config#339 has merged, let's give it a try. (That said until we start pinning cosa in the FCOS pipeline this could start failing on the prod pipelines that won't have that change) |
I initially thought we could just scrape the text serial console but the ISO doesn't output there by default 😢 Instead, this now depends on coreos/fedora-coreos-config#339
024ae70 to
64bd888
Compare
|
🎉 |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ashcrow, cgwalters The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
I initially thought we could just scrape the text serial
console but the ISO doesn't output there by default cry
Instead, this now depends on coreos/fedora-coreos-config#339