-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Implement core grains support for Linux ppc64le #60519
Conversation
264af20
to
65dbb04
Compare
As this PR adds support for reading arbitrary Device Tree data into grains, I've tested this on Raspberry PI as well (as ARM also has DT subsystem):
So following grains will get set for RPi devices:
That will presumably work for all ARM devices / all devices that have Device Tree subsystem, I don't have any other I could test that on. I've tried to spawn AWS Graviton 2 instance, minion runs there fine (had to rebuild packages, it's aarch64), but it doesn't have DT, it rather has SMBIOS 🤷♂️ FYI @waynew |
65dbb04
to
b6a1c1a
Compare
FYI,
(it's very similar to RPi 4 - no |
b6a1c1a
to
a93db95
Compare
FYI, this is how this MR looks like for QEMU virtualized ppc64el Ubuntu:
Grains:
|
Hi! I'm your friendly PR bot!You might be wondering what I'm doing commenting here on your PR. Yes, as a matter of fact, I am... I'm just here to help us improve the documentation. I can't respond to Okay... so what do you do? I detect modules that are missing docstrings or "CLI Example" on existing docstrings! So what does that have to do with my PR? I noticed that in this PR there are some files changed that have some of these Okay, what are they? Well, my favorite, is that since you were making changes here I'm hoping that If I can, then what? Well, you can either add them to this PR or add them to another PR. Either way is fine! Well... what if I can't, or don't want to? That's also fine! We appreciate all contributions to the Salt Project. If you Whatever approach you decide to take, just drop a comment here letting us know! Detected Issues (click me)[INFO] Initializing environment for https://github.com/saltstack/pip-tools-compile-impersonate. [INFO] Initializing environment for https://github.com/asottile/pyupgrade. [INFO] Initializing environment for https://github.com/saltstack/pre-commit-remove-import-headers. [INFO] Initializing environment for https://github.com/s0undt3ch/salt-rewrite. [INFO] Initializing environment for https://github.com/timothycrosley/isort. [INFO] Initializing environment for https://github.com/timothycrosley/isort:toml. [INFO] Initializing environment for https://github.com/psf/black. [INFO] Initializing environment for https://github.com/asottile/blacken-docs. [INFO] Initializing environment for https://github.com/asottile/blacken-docs:black==21.7b0. [INFO] Initializing environment for https://github.com/PyCQA/bandit. [INFO] Initializing environment for https://github.com/saltstack/invoke-pre-commit. [INFO] Initializing environment for https://github.com/saltstack/invoke-pre-commit:blessings,distro,jinja2,msgpack,pyyaml. [INFO] Initializing environment for https://github.com/saltstack/mirrors-nox. [INFO] Initializing environment for https://github.com/saltstack/mirrors-nox:pip>=20.2.4,<21.2,setuptools<58.0. [INFO] Installing environment for https://github.com/saltstack/invoke-pre-commit. [INFO] Once installed this environment will be reused. [INFO] This may take a few minutes... Check Known Missing Docstrings...........................................Passed - hook id: invoke - duration: 1.27s Thanks again! |
@dhiltonp is there any way to get this forward? it's more or less 4 months since I've created this PR with no reply from core team, thanks |
9160dd9
to
7417b31
Compare
What does this PR do?
Implements support for core grains for Linux ppc64le architecture.
More specifically:
Side note - there seems to be DT uuid attribute, it's not in specs and I don't have any machine currently with this set, so uuid grain is ommited
The tests (especially the
test_linux_devicetree_data
) are still a little bit sketchy, so I will try to make them a bit better.What issues does this PR fix or reference?
Related to #60518
Previous Behavior
Grains that rely on dmidecode (productname, manufacturer, serialnumber) are missing completely
New Behavior
Merge requirements satisfied?
Commits signed with GPG?
Yes