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

Fix curl failed #1094

Merged
merged 1 commit into from
Jan 15, 2024
Merged

Fix curl failed #1094

merged 1 commit into from
Jan 15, 2024

Conversation

weidongkx
Copy link
Contributor

@weidongkx weidongkx commented Jan 15, 2024 •
edited by SuperQ
Loading

Fixes: #1093

curl version is 7.61.1  in rhel8.8,
and --no-progress-meter is added in 7.67.0.
so there is error curl: option --no-progress-meter: is unknown

For better compatibility, we shoule use the option -sS

@wd3993
Fix curl failed
e9b623b 
```
curl version is 7.61.1  in rhel8.8,
and --no-progress-meter is added in 7.67.0.
so there is error curl: option --no-progress-meter: is unknown

For better compatibility, we shoule use the option -sS
```

Signed-off-by: weidongkl <[email protected]>
@weidongkx
Copy link
Contributor Author

curl man page says:

-s, --silent

         Silent or quiet mode. Don't show progress meter  or  error  mes‐
         sages.   Makes  Curl mute. It will still output the data you ask
         for, potentially even to the terminal/stdout unless you redirect
         it.

         Use  -S,  --show-error  in  addition  to  this option to disable
         progress meter but still show error messages.

@SuperQ SuperQ merged commit fdc64b3 into prometheus:main Jan 15, 2024
6 checks passed
harshavmb pushed a commit to harshavmb/snmp_exporter that referenced this pull request Feb 16, 2024
@weidongkx @harshavmb
Fix curl failed (prometheus#1094)
44601d8 
```
curl version is 7.61.1  in rhel8.8,
and --no-progress-meter is added in 7.67.0.
so there is error curl: option --no-progress-meter: is unknown

For better compatibility, we shoule use the option -sS
```

Signed-off-by: weidongkl <[email protected]>
Signed-off-by: Harshavardhan Musanalli <[email protected]>
@SuperQ SuperQ mentioned this pull request May 10, 2024
Merged
SuperQ pushed a commit that referenced this pull request May 11, 2024
@bastischubert
Release v0.26.0 (#1181)
44f8732 
* [CHANGE] Improve generator parse error handling #1167
* [ENHANCEMENT] generator: Add generator HELP override #1106
* [ENHANCEMENT] Refactoring of Scrape process, fixing multiple module issues #1111
* [ENHANCEMENT] Skip using an interactive terminal in "make docker-generate". #1113
* [ENHANCEMENT] Add SNMPInflight metric #1119
* [FEATURE] Support for passing username, password & priv_password as env vars #1074
* [FEATURE] Add GoSNMP logger #1157
* [FEATURE] Add a "snmp_context" parameter to the URL #1163
* [BUGFIX] generator: curl failed #1094
* [BUGFIX] Fix SNMPv3 password configuration #1122
* [BUGFIX] generator: Update generator User-Agent #1133
* [BUGFIX] generator: fix mibs directory specification for parse_errors command #1135
* [BUGFIX] generator: remove extra character from dell iDrac-SMIv1 MIB #1141
* [BUGFIX] Fix do not expand envvars for empty config fields #1148

snmp.yml changes:
* Updated Cisco MIBs #1180
* Updated Cyberpower MIBs #1124
* Updated servertech_sentry3 #1090
* Added support for Dell iDrac  #1125
---------

Signed-off-by: Sebastian Schubert <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SuperQ SuperQ SuperQ approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

curl: option --no-progress-meter: is unknown
3 participants
@weidongkx @SuperQ @wd3993