Skip to content

Add multi-console log support#11

Merged
StykMartin merged 1 commit intobeaker-project:release-27from
jvillal-amp:release-27
Apr 27, 2020
Merged

Add multi-console log support#11
StykMartin merged 1 commit intobeaker-project:release-27from
jvillal-amp:release-27

Conversation

@jvillal-amp
Copy link
Copy Markdown
Contributor

Support console files with the name of $CONSOLE_LOGS/$FQDN*

For example:
If CONSOLE_LOGS=/var/consoles and the FQDN=test.example.com

Then the following files will be logged as console files:
/var/consoles/test.example.com -> console.log
/var/consoles/test.example.com-bmc -> console-bmc.log
/var/consoles/test.example.com-serial2 -> console-serial2.log

Bug: 1771666

Support console files with the name of $CONSOLE_LOGS/$FQDN*

For example:
  If CONSOLE_LOGS=/var/consoles and the FQDN=test.example.com

  Then the following files will be logged as console files:
    /var/consoles/test.example.com -> console.log
    /var/consoles/test.example.com-bmc -> console-bmc.log
    /var/consoles/test.example.com-serial2 -> console-serial2.log

Bug: 1771666
@StykMartin
Copy link
Copy Markdown
Contributor

StykMartin commented Apr 7, 2020

/packit build /test 👀

@packit-as-a-service
Copy link
Copy Markdown

Congratulations! One of the builds has completed. 🍾

You can install the built RPMs by following these steps:

  • sudo yum install -y dnf-plugins-core on RHEL 8
  • sudo dnf install -y dnf-plugins-core on Fedora
  • dnf copr enable packit/beaker-project-beaker-11
  • And now you can install the packages.

Please note that the RPMs should be used only in a testing environment.

@StykMartin StykMartin merged commit f8748f4 into beaker-project:release-27 Apr 27, 2020
Copy link
Copy Markdown
Contributor

@StykMartin StykMartin left a comment

Choose a reason for hiding this comment

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

Thanks @jvillal-amp for your contribution. Everything looks ok.

@jvillal-amp
Copy link
Copy Markdown
Contributor Author

Thanks @jvillal-amp for your contribution. Everything looks ok.

Thanks @StykMartin for the review! I hope it will be useful for other people too 😊

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.

2 participants