Skip to content

Commit

Permalink
Merge pull request #7692 from pepesi/master
Browse files Browse the repository at this point in the history
fix flannel-migration-controller failed
  • Loading branch information
marvin-tigera authored May 24, 2023
2 parents 145a63d + 3e76a25 commit 8372da5
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -298,7 +298,7 @@ func (c *flannelMigrationController) readAndUpdateFlannelEnvConfig() error {
// Convert subnet.env content to json string and update flannel-migration-config ConfigMap.
// So that it could be populated into migration controller pod next time it starts.
val := strings.Replace(data, "\n", ";", -1)
err = updateConfigMapValue(c.k8sClientset, c.config.FlannelDaemonsetNamespace, migrationConfigMapName, migrationConfigMapEnvKey, val)
err = updateConfigMapValue(c.k8sClientset, c.config.CalicoDaemonsetNamespace, migrationConfigMapName, migrationConfigMapEnvKey, val)
if err != nil {
return err
}
Expand Down Expand Up @@ -355,7 +355,7 @@ func (c *flannelMigrationController) CheckShouldMigrate() (bool, error) {

// Update calico-config ConfigMap veth_mtu.
// So that it could be populated into calico-node pods.
err = updateConfigMapValue(c.k8sClientset, c.config.FlannelDaemonsetNamespace, calicoConfigMapName, calicoConfigMapMtuKey, fmt.Sprintf("%d", c.config.FlannelMTU))
err = updateConfigMapValue(c.k8sClientset, c.config.CalicoDaemonsetNamespace, calicoConfigMapName, calicoConfigMapMtuKey, fmt.Sprintf("%d", c.config.FlannelMTU))
if err != nil {
return false, err
}
Expand Down

0 comments on commit 8372da5

Please sign in to comment.