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

Develop bmc-log-collector and update machines-endpoints #1420

Closed
wants to merge 5 commits into from

Conversation

takara9
Copy link
Contributor

@takara9 takara9 commented Sep 9, 2024

Recreate PR based on #1361, #1389

@takara9 takara9 self-assigned this Sep 9, 2024
@takara9 takara9 requested review from masa213f and zoetrope and removed request for zoetrope and masa213f September 9, 2024 06:43
delete unnecessary coment

update TAG and go version
bmc-log-collector/Makefile Outdated Show resolved Hide resolved

## Usage

*** under writing ***
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you write here.

bmc-log-collector/bmc-user_test.go Outdated Show resolved Hide resolved
Comment on lines +98 to +100



Copy link
Contributor

Choose a reason for hiding this comment

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

There are three blank lines. Is this intentional?

Comment on lines +110 to +112



Copy link
Contributor

Choose a reason for hiding this comment

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

There are three blank lines. Is this intentional?

bmc-log-collector/log-collector.go Outdated Show resolved Hide resolved
bmc-log-collector/log-pointer.go Outdated Show resolved Hide resolved
bmc-log-collector/log-pointer.go Outdated Show resolved Hide resolved
bmc-log-collector/log-pointer.go Outdated Show resolved Hide resolved
machines-endpoints/pkg/machines-endpoints/main.go Outdated Show resolved Hide resolved
bmc-log-collector/main.go Outdated Show resolved Hide resolved
@takara9 takara9 marked this pull request as draft September 17, 2024 07:39
@takara9 takara9 closed this Sep 25, 2024
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