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

lack of support macos not supported #1104

Open
ssbarnea opened this issue Apr 8, 2022 · 9 comments
Open

lack of support macos not supported #1104

ssbarnea opened this issue Apr 8, 2022 · 9 comments
Labels
help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. kind/bug Categorizes issue or PR as related to a bug. lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. priority/backlog Higher priority than priority/awaiting-more-evidence. sig/user-experience Issues or PRs related to the User Experience of our Services, Tools, and Libraries. triage/accepted Indicates an issue or PR is ready to be actively worked on. triage/needs-information Indicates an issue needs more information in order to work on it.

Comments

@ssbarnea
Copy link

ssbarnea commented Apr 8, 2022

I only understood much later that this tool works only on Linux as that was not obvious from its description.

$ thamos config         
2022-04-08 14:34:58,016 [61403] INFO     thamos: No configuration file found, creating one from a configuration template from a default template
2022-04-08 14:34:58,016 [61403] INFO     thamos.config: Discovering host runtime environment
2022-04-08 14:34:58,016 [61403] ERROR    thamos.discover: Failed to obtain CPU specific information: [Errno 2] No such file or directory: '/proc/cpuinfo'
Traceback (most recent call last):
  File "/Users/ssbarnea/.pyenv/versions/3.10.3/lib/python3.10/site-packages/thamos/discover.py", line 98, in discover_cpu
    with open(_PROC_CPU_INFO, "r") as cpu_info_file:
FileNotFoundError: [Errno 2] No such file or directory: '/proc/cpuinfo'
/bin/sh: edit: command not found
2022-04-08 14:34:58,044 [61403] ERROR    thamos: edit: Editing failed
FAIL: 1

The reality is that most people using python expect to see tools working on any POSIX system, including MacOS. If you happen not use have/use a mac, you can use BSD as close enough system to test against, as if you get it working on BSD, it will almost for sure work on MacOS.

If there is no desire to make it work on macos, at least Linux should be mentioned as part of the first sentence in product description, if not even the title itself. Also, adding a nice error message at cli would provide a better experience than a delayed tracedump.

@ssbarnea ssbarnea added the kind/bug Categorizes issue or PR as related to a bug. label Apr 8, 2022
@mayaCostantini
Copy link
Contributor

/sig user-experience

@sesheta sesheta added the sig/user-experience Issues or PRs related to the User Experience of our Services, Tools, and Libraries. label Apr 11, 2022
@fridex
Copy link
Contributor

fridex commented Apr 11, 2022

We could catch the FileNotFoundError and produce a warning in these cases (and set CPU information to None). I'm not familiar with obtaining CPU info on macos (I don't have macos env to test this). This could be a good-enough solution to support calling advises on macos.

@goern goern added the triage/needs-information Indicates an issue needs more information in order to work on it. label Apr 25, 2022
@fridex
Copy link
Contributor

fridex commented May 2, 2022

It looks like the config command works as expected - it reports that there was an error loading /proc/cpuinfo and keeps CPU information set to None. It looks like the following line is the culprint:

/bin/sh: edit: command not found

We use click.edit (see docs) to open the config file. However, it fails to find edit editor to open the configuration file.

@sesheta
Copy link
Member

sesheta commented Jul 31, 2022

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

/lifecycle stale

@sesheta sesheta added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Jul 31, 2022
@codificat
Copy link
Member

/remove-lifecycle stale

Suggested action plan:

  • document what are the options to run thamos on MacOS (e.g. getting thamos advise for a linux-based stack)
  • handle the situation and point to the doc above instead of throwing a traceback

@sesheta sesheta removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Aug 8, 2022
@codificat
Copy link
Member

/priority backlog
/triage accepted

@sesheta sesheta added priority/backlog Higher priority than priority/awaiting-more-evidence. triage/accepted Indicates an issue or PR is ready to be actively worked on. labels Aug 8, 2022
@codificat
Copy link
Member

/help
we could benefit from MacOS users contributions here

@sesheta
Copy link
Member

sesheta commented Aug 8, 2022

@codificat:
This request has been marked as needing help from a contributor.

Please ensure the request meets the requirements listed here.

If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-help command.

In response to this:

/help
we could benefit from MacOS users contributions here

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@sesheta sesheta added the help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. label Aug 8, 2022
@codificat
Copy link
Member

/lifecycle frozen

@sesheta sesheta added the lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. label Sep 19, 2022
@codificat codificat moved this from 🆕 New to 📋 Backlog in Planning Board Sep 26, 2022
@codificat codificat moved this to 🆕 New in Planning Board Sep 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. kind/bug Categorizes issue or PR as related to a bug. lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. priority/backlog Higher priority than priority/awaiting-more-evidence. sig/user-experience Issues or PRs related to the User Experience of our Services, Tools, and Libraries. triage/accepted Indicates an issue or PR is ready to be actively worked on. triage/needs-information Indicates an issue needs more information in order to work on it.
Projects
Status: 📋 Backlog
Development

No branches or pull requests

6 participants