Skip to content

Vendor gopsutil with go mod fix#8108

Merged
notnoop merged 1 commit intomasterfrom
c-gopsutil-go-mod-fork
Jun 3, 2020
Merged

Vendor gopsutil with go mod fix#8108
notnoop merged 1 commit intomasterfrom
c-gopsutil-go-mod-fork

Conversation

@notnoop
Copy link
Copy Markdown
Contributor

@notnoop notnoop commented Jun 3, 2020

Point to gopsutil fork with shirou/gopsutil#889 changes. I used hashicorp/gopsutil rather than pointing to https://github.com/Lomanic/gopsutil/tree/issue832 directly just in case..

@notnoop notnoop requested a review from shoenig June 3, 2020 19:14
@shoenig
Copy link
Copy Markdown
Contributor

shoenig commented Jun 3, 2020

LGTM, would like to rerun the tests to get past transient failures. gopsutil always has an extra surprise ready

Base automatically changed from vendor-sync-20200603 to master June 3, 2020 19:34
@notnoop notnoop force-pushed the c-gopsutil-go-mod-fork branch from f5b8fc7 to c62a19a Compare June 3, 2020 19:35
@notnoop
Copy link
Copy Markdown
Contributor Author

notnoop commented Jun 3, 2020

I rebased to account for #8107 being merged.

@notnoop notnoop merged commit fa845c7 into master Jun 3, 2020
@notnoop notnoop deleted the c-gopsutil-go-mod-fork branch June 3, 2020 20:09
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 3, 2023

I'm going to lock this pull request because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants