starred/proxmox-api-go
1
0
Fork 0
mirror of https://github.com/Telmate/proxmox-api-go.git synced 2026-04-26 07:55:50 +03:00
Code Issues 62 Projects Releases Packages Wiki Activity

[GH-ISSUE #168] config file prompted even for targets that dont require config #36

New issue
Closed
opened 2026-02-28 00:40:16 +03:00 by kerem · 3 comments
kerem commented 2026-02-28 00:40:16 +03:00
Owner
Copy link

Originally created by @thorntonmc on GitHub (Apr 26, 2022).
Original GitHub issue: https://github.com/Telmate/proxmox-api-go/issues/168

#164 created the following issue for me.

Part of my workflow involves getting a list of all VMs. This does not normally require a config file, however since GetConfig is always called now, it expects some sort of config to be directed in from stdin (which hangs indefinitely if nothing is directed) .

Is this intended? If not, I can create the PR to resolve this issue -- I just wanted to make sure if this was the intended workflow or not.

Originally created by @thorntonmc on GitHub (Apr 26, 2022). Original GitHub issue: https://github.com/Telmate/proxmox-api-go/issues/168 #164 created the following issue for me. Part of my workflow involves getting a list of all VMs. This does not normally require a config file, however since `GetConfig` is always called now, it expects some sort of config to be directed in from stdin (which hangs indefinitely if nothing is directed) . Is this intended? If not, I can create the PR to resolve this issue -- I just wanted to make sure if this was the intended workflow or not.
kerem closed this issue 2026-02-28 00:40:16 +03:00
kerem commented 2026-02-28 00:40:17 +03:00
Author
Owner
Copy link

@Tinyblargon commented on GitHub (Apr 26, 2022):

Thanks for reporting this.

This behavior was not intended when i made the change.
Currently the --file flag and os.stdin are only intended to supply a config when a item gets set/created/updated.

<!-- gh-comment-id:1109203100 --> @Tinyblargon commented on GitHub (Apr 26, 2022): Thanks for reporting this. This behavior was not intended when i made the change. Currently the --file flag and os.stdin are only intended to supply a config when a item gets set/created/updated.
kerem commented 2026-02-28 00:40:17 +03:00
Author
Owner
Copy link

@thorntonmc commented on GitHub (Apr 26, 2022):

Thanks @Tinyblargon - I'll throw up a PR and @ you on it.

<!-- gh-comment-id:1109208495 --> @thorntonmc commented on GitHub (Apr 26, 2022): Thanks @Tinyblargon - I'll throw up a PR and @ you on it.
kerem commented 2026-02-28 00:40:17 +03:00
Author
Owner
Copy link

@mleone87 commented on GitHub (Apr 28, 2022):

closing it if it's working as expected now

<!-- gh-comment-id:1111970974 --> @mleone87 commented on GitHub (Apr 28, 2022): closing it if it's working as expected now
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
master
dependabot/go_modules/golang.org/x/crypto-0.45.0
issue-300-fix
go-119-upgrade
logic-improvements
getUserPermissions
cicustom
next_if_conflict
cloud-init
add_full_clone_option
No results found.
Labels
Clear labels   
good first issue

   
issue/confirmed

   
issue/critical

   
proposal/accepted

   
pull-request

Mirrored from GitHub Pull Request

  
type/bug

   
type/enhancement

   
type/feature

   
type/question

   
type/refactoring

   
type/testing

   
type/testing
No labels
good first issue
issue/confirmed
issue/critical
proposal/accepted
pull-request
type/bug
type/enhancement
type/feature
type/question
type/refactoring
type/testing
type/testing
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/proxmox-api-go#36
No description provided.