We previously changed "subnet" to "pdn" in smf and upf conf files due to schema changing, but one instance of "pdn" still exists in update_smf. As currently written, every time we call colteconf update it just adds a value to the subnet field instead of clearing out the old values.
LGTM, same comment at the other PR about the commit message. Your text here in the PR is great, but that doesn't actually make it into the commit on merge.
We previously changed "subnet" to "pdn" in smf and upf conf files due to schema changing, but one instance of "pdn" still exists in update_smf. As currently written, every time we call colteconf update it just adds a value to the subnet field instead of clearing out the old values.