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

system_info table: add "board_xxx" columns like "hardware_xxx" #6398

Merged
merged 3 commits into from
Apr 18, 2020

Conversation

jychen7
Copy link
Contributor

@jychen7 jychen7 commented Apr 16, 2020

It turns out hardware_xxx is not useful in SuperMicro servers

hardware_model
Actual: Super Server
Expect: X11SSH-F

hardware_version
Actual: 0123456789
Expect: 1.01

hardware_serial
Actual: 0123456789
Expect: WM199xxxxxxx

To submit a PR please make sure to follow the next steps:

  • Read the CONTRIBUTING.md guide on the root of the repo.
  • Ensure the code is formatted building the format_check target,
    if not move the committed files to the stage area,
    build the format target to format, then re-commit.
    More information is available on the wiki.
  • Ensure your PR contains a single logical change.
  • Ensure your PR contains tests for the changes you're submitting.
  • Describe your changes with as much detail as you can.

It turns out hardware_xxx is not useful in SuperMicro servers
hardware_model
Actual: Super Server
Expect: X11SSH-F

hardware_version
Actual: 0123456789
Expect: 1.01

hardware_serial
Actual: 0123456789
Expect: WM199xxxxxxx
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Apr 16, 2020

CLA Check
The committers are authorized under a signed CLA.

@theopolis
Copy link
Member

Thanks for this! The next step is signing the CLA.

@jychen7
Copy link
Contributor Author

jychen7 commented Apr 17, 2020

Thanks for this! The next step is signing the CLA.

@theopolis Is there way to sign as individual? It shows the corporate option only, which I need confirm with company before doing it
done, it looks like the footer just hide the "Individual" option, because of my 11 screen

@theopolis theopolis merged commit a851d85 into osquery:master Apr 18, 2020
@jychen7
Copy link
Contributor Author

jychen7 commented Apr 19, 2020

nice, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants