Don't overwrite hardcoded values #4
Labels
No Label
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
No Milestone
No project
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: Sysdeploy/idlers-agent#4
Loading…
Reference in New Issue
Block a user
No description provided.
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Keys like provider-id, location-id, are hard coded in the agent. This would overwrite the manually adjusted values in the app. Similarly, for RAM, reading it from
/proc/meminfo
doesn't show the amount of physical RAM available, but only that available outside of the kernel. So I don't want this overwritten, but at the same time when the amount of RAM changes, this should be reflected. So we should default to calculating RAM fromdmidecode
as that's accurate.As per discussion in Skype, we should split the keys into two categories: those that should be set only during creation of a server record, and those that should be set either during creation or during an update. Things like serial numbers or primary IP (things that can be accurately determined by the agent and may change more frequently) are a good example of keys that are always updated, and the secondary IPs, name of the provider, how much it costs per month are a good example of keys that shouldn't be updated (the agent doesn't know how to obtain them, and they probably don't change that often).
Probably this can be closed? @PeterSurda
Fixed in #7