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.

Sign up for GitHub

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

Jump to bottom

salt.modules.status mostly broken on solaris like operating systems. #30493

Closed
sjorge opened this issue Jan 21, 2016 · 6 comments
Closed

salt.modules.status mostly broken on solaris like operating systems. #30493

sjorge opened this issue Jan 21, 2016 · 6 comments
Labels
Execution-Module Feature new functionality including changes to functionality and code refactors, etc. Platform Relates to OS, containers, platform-based utilities like FS, system based apps
Milestone
Approved

Comments

@sjorge
Copy link
Contributor

sjorge commented Jan 21, 2016

This should probably not be loaded, only loadavg seems to work and some even dump stacktraces like diskusage.

I'll eventually get around to fixing this.
@jfindlay jfindlay changed the title salt.modules.states mostly broken on solaris like operating systems. salt.modules.status mostly broken on solaris like operating systems. Jan 21, 2016
@jfindlay jfindlay added Feature new functionality including changes to functionality and code refactors, etc. Execution-Module Platform Relates to OS, containers, platform-based utilities like FS, system based apps labels Jan 21, 2016
@jfindlay jfindlay added this to the Approved milestone Jan 21, 2016
@jfindlay
Copy link
Contributor

@sjorge, thanks for the report.

@sjorge
Copy link
Contributor Author

sjorge commented Jan 21, 2016

Quick questions, is there a reason why status.netstat does not simply call network.netstat? (the later supports more operating systems like OpenBSD, Solaris, ... as I fixed that one a while ago)

@jfindlay
Copy link
Contributor

@sjorge, that is a good idea. You should change it.

@sjorge
Copy link
Contributor Author

sjorge commented Jan 21, 2016

I'll try to fix up most of these when I have some time. I'll do that one first then since it is an easy fix :3

~ sjorge

On 21 Jan 2016, at 23:34, Justin Findlay notifications@github.com wrote:

@sjorge, that is a good idea. You should change it.

—
Reply to this email directly or view it on GitHub.

@sjorge
Copy link
Contributor Author

sjorge commented Jun 25, 2016

As for status.netstats calling network.netstat... that is a no go. The data fetched from are different.

@sjorge sjorge mentioned this issue Jun 25, 2016
Merged
@sjorge
Copy link
Contributor Author

sjorge commented Oct 4, 2016

Most of this was addressed late June.

@sjorge sjorge closed this as completed Oct 4, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Execution-Module Feature new functionality including changes to functionality and code refactors, etc. Platform Relates to OS, containers, platform-based utilities like FS, system based apps
Projects
None yet
Milestone
Approved
Development

No branches or pull requests
2 participants
@sjorge @jfindlay