Page MenuHomeVyOS Platform
Create Task
Maniphest T6058

Commit-Archive Save doesn't use https_proxy
Open, NormalPublicBUG
Actions

Description

Hey,

we tried to use the git+https commit-archive backend, but were unable to use it as we need to use a proxy for outside communication.
The reason for this is that the git subprocess is being called without the https_proxy environment variable that we set in the vyos config.

I found the lines where the environment for git were defined and when I added the correct environment var the commits were able to be saved in git.
https://github.com/vyos/vyos-1x/blob/72e597d5c258b1730d59d9f8cd08331338cee5b5/python/vyos/remote.py#L361-L367

Thanks!

Best regards,
modzilla

Details

Difficulty level
Easy (less than an hour)
Version
ep1
Why the issue appeared?
Will be filled on close
Is it a breaking change?
Perfectly compatible
Issue type
Bug (incorrect behavior)

Event Timeline

modzilla99 created this task.Feb 23 2024, 12:49 PM
Viacheslav triaged this task as Normal priority.Feb 23 2024, 12:59 PM
Viacheslav edited projects, added VyOS 1.4 Sagitta (1.4.0-epa1), VyOS 1.5 Circinus; removed VyOS 1.4 Sagitta.
Viacheslav changed the task status from Open to Needs reporter action.Apr 23 2024, 5:09 PM
Viacheslav subscribed.

@modzilla99 Could you provide an example of set commands to reproduce?

pasik subscribed.Apr 23 2024, 6:27 PM
syncer edited projects, added VyOS 1.4 Sagitta (1.4.0-GA); removed VyOS 1.4 Sagitta (1.4.0-epa1).May 15 2024, 9:30 AM
modzilla99 added a comment.May 16 2024, 7:42 AM

Hey,

sorry for the late reply.You don't need any special commands. The only thing you have to set is the proxy and the commit archive.

set system config-management commit-archive location 'git+https://vyos:[email protected]/infrastructure/vyos-configuration'
set system proxy port '8888'
set system proxy url 'http://10.7.2.100'

Best regards,
Justin Lamp

Viacheslav changed the task status from Needs reporter action to Open.May 16 2024, 7:55 AM
dmbaturin edited projects, added VyOS 1.4 Sagitta (1.4.1); removed VyOS 1.4 Sagitta (1.4.0-GA), VyOS 1.5 Circinus.Jul 2 2024, 7:29 PM
dmbaturin added a project: Restricted Project.Sep 15 2024, 5:03 PM