add comments

pkg/plan/embeddedcluster.go

@@ -52,6 +52,7 @@ func executeECUpgrade(s store.Store, p *types.Plan, step *types.PlanStep) error
 		return errors.Wrap(err, "failed to load kotskinds from path")
 	}
 
+	// TODO (@salah): old kots should not be constructing the new installation object
 	newInstall := &ecv1beta1.Installation{
 		TypeMeta: metav1.TypeMeta{
 			APIVersion: ecv1beta1.GroupVersion.String(),

pkg/plan/plan.go

@@ -53,6 +53,10 @@ func PlanUpgrade(s store.Store, opts PlanUpgradeOptions) (*types.Plan, error) {
 		Owner:             types.StepOwnerKOTS,
 	})
 
+	// TODO (@salah): add a step to upgrade kots here
+	// to make sure later steps are executed by the new version,
+	// as old kots might not be able to parse new kots kinds (and EC config)
+
 	// embedded cluster upgrade
 	if ecVersion != util.EmbeddedClusterVersion() {
 		plan.Steps = append(plan.Steps, &types.PlanStep{