Skip to content
This repository has been archived by the owner on Aug 12, 2022. It is now read-only.

fix: Use 'cur' ulimit in calculation, not 'max' #399

Merged
merged 1 commit into from
Jul 3, 2022
Merged

fix: Use 'cur' ulimit in calculation, not 'max' #399

merged 1 commit into from
Jul 3, 2022

Conversation

shimonp21
Copy link
Contributor

The 'cur' ulimit is the actual limit on num-open-files enforced by the kernel.

Summary


Use the following steps to ensure your PR is ready to be reviewed

  • Read the contribution guidelines πŸ§‘β€πŸŽ“
  • Run go fmt to format your code πŸ–Š
  • Lint your changes via golangci-lint run 🚨 (install golangci-lint here)
  • Update or add tests πŸ§ͺ
  • Ensure the status checks below are successful βœ…

The 'cur' ulimit is the actual limit on num-open-files enforced by the kernel.
@shimonp21 shimonp21 requested a review from a team as a code owner July 1, 2022 11:25
@shimonp21 shimonp21 requested review from roneli, erezrokah and a team and removed request for a team July 1, 2022 11:25
@erezrokah erezrokah merged commit 1acc3de into cloudquery:main Jul 3, 2022
kodiakhq bot pushed a commit that referenced this pull request Jul 3, 2022
πŸ€– I have created a release *beep* *boop*
---


## [0.13.2](v0.13.1...v0.13.2) (2022-07-03)


### Bug Fixes

* Use 'cur' ulimit in calculation, not 'max' ([#399](#399)) ([1acc3de](1acc3de))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
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