Project

General

Profile

Actions
Copy link

Bug #11284

closed

Warn during inventory if sending through http not https

Added by Alexis Mousset over 7 years ago. Updated over 6 years ago.

Status:
Released
Priority:
N/A
Assignee:
Alexis Mousset
Category:
Packaging
Target version:
4.1.13
Pull Request:
https://github.com/Normation/rudder-t...
Severity:
Minor - inconvenience | misleading | easy workaround
UX impact:
User visibility:
Operational - other Techniques | Technique editor | Rudder settings
Effort required:
Small
Priority:
41
Name check:
Fix check:
Regression:

Description

Tested on Debian stretch, but should happen elsewhere. After installing rudder-agent in a light LXC template, inventory faisl because curl is missing. It may be a detection issue (or a missing dependency).

Actions
Copy link
 #1

Updated by François ARMAND over 7 years ago

  • Target version set to 3.1.23
  • Severity set to Major - prevents use of part of Rudder | no simple workaround
  • User visibility set to Getting started - demo | first install | level 1 Techniques
  • Priority changed from 0 to 70

Setting it to "major", because there is no error message (or better, not easily understanding error message for the user). So the user need to understand that the problem is the missing dependency to curl.
I'm setting it to "getting started", because it may be that a standardize homegrown VM doesn't come with curl.

Actions
Copy link
 #3

Updated by François ARMAND over 7 years ago

  • Assignee set to Vincent MEMBRÉ
Actions
Copy link
 #4

Updated by Vincent MEMBRÉ over 7 years ago

  • Target version changed from 3.1.23 to 3.1.24
Actions
Copy link
 #5

Updated by Benoît PECCATTE about 7 years ago

  • Effort required set to Small
  • Priority changed from 70 to 73
Actions
Copy link
 #6

Updated by Benoît PECCATTE about 7 years ago

  • Priority changed from 73 to 72

Solution: print an error message.
Then open an issue to emmbbed or depend on curl for the agent.

Actions
Copy link
 #7

Updated by Benoît PECCATTE about 7 years ago

  • Assignee changed from Vincent MEMBRÉ to Alexis Mousset
Actions
Copy link
 #8

Updated by François ARMAND about 7 years ago

  • Subject changed from Inventory fails is curl is not installed to Inventory fails if curl is not installed
Actions
Copy link
 #9

Updated by Vincent MEMBRÉ about 7 years ago

  • Target version changed from 3.1.24 to 3.1.25
Actions
Copy link
 #10

Updated by Benoît PECCATTE about 7 years ago

  • Priority changed from 72 to 83
Actions
Copy link
 #11

Updated by Benoît PECCATTE about 7 years ago

  • Target version changed from 3.1.25 to 4.1.9
Actions
Copy link
 #12

Updated by Benoît PECCATTE about 7 years ago

  • Subject changed from Inventory fails if curl is not installed to Warn during inventory if curl is not installed
Actions
Copy link
 #13

Updated by Vincent MEMBRÉ about 7 years ago

  • Target version changed from 4.1.9 to 4.1.10
  • Priority changed from 83 to 82
Actions
Copy link
 #14

Updated by Alexis Mousset about 7 years ago

  • Status changed from New to In progress
  • Priority changed from 82 to 81
Actions
Copy link
 #15

Updated by Vincent MEMBRÉ almost 7 years ago

  • Target version changed from 4.1.10 to 4.1.11
  • Priority changed from 81 to 78
Actions
Copy link
 #16

Updated by Alexis Mousset over 6 years ago

  • Status changed from In progress to New
  • Priority changed from 78 to 76
Actions
Copy link
 #17

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 4.1.11 to 4.1.12
Actions
Copy link
 #18

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 4.1.12 to 4.1.13
  • Priority changed from 76 to 75
Actions
Copy link
 #19

Updated by Benoît PECCATTE over 6 years ago

  • Target version changed from 4.1.13 to 411
Actions
Copy link
 #20

Updated by Benoît PECCATTE over 6 years ago

  • Target version changed from 411 to 4.1.13
Actions
Copy link
 #21

Updated by Benoît PECCATTE over 6 years ago

  • Subject changed from Warn during inventory if curl is not installed to Warn during inventory if sending through http not https
  • Status changed from New to Rejected
  • Priority changed from 75 to 73
Actions
Copy link
 #22

Updated by Benoît PECCATTE over 6 years ago

  • Severity changed from Major - prevents use of part of Rudder | no simple workaround to Minor - inconvenience | misleading | easy workaround
  • Priority changed from 73 to 57
Actions
Copy link
 #23

Updated by Benoît PECCATTE over 6 years ago

  • Status changed from Rejected to New
  • User visibility changed from Getting started - demo | first install | level 1 Techniques to Operational - other Techniques | Technique editor | Rudder settings
  • Priority changed from 57 to 41

Now we fallback to minicurl and http when curl is not install this is not a problem anymore see #12155
Moreover curl always installed in 4.3

Actions
Copy link
 #24

Updated by Benoît PECCATTE over 6 years ago

  • Status changed from New to In progress
  • Assignee changed from Alexis Mousset to Benoît PECCATTE
Actions
Copy link
 #25

Updated by Benoît PECCATTE over 6 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Benoît PECCATTE to Alexis Mousset
  • Pull Request set to https://github.com/Normation/rudder-techniques/pull/1318
Actions
Copy link
 #26

Updated by Benoît PECCATTE over 6 years ago

  • Status changed from Pending technical review to Pending release
Actions
Copy link
 #27

Updated by Vincent MEMBRÉ over 6 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 4.1.13, 4.2.7 and 4.3.3 which were released today.

Actions
Copy link

Also available in: Atom PDF