Add comments indicating this is a short-term workaround only

Author: johnsca

juju/bundle.py

@@ -329,6 +329,9 @@ async def run(self, context):
         :param context: is used for any methods or properties required to
             perform a change.
         """
+        # NB: this should really be handled by the controller when generating the
+        # bundle change plan, and this short-term workaround may be missing some
+        # aspects of the logic which the CLI client contains to handle edge cases.
         if self.application in context.model.applications:
             log.debug('Skipping %s; already in model', self.application)
             return self.application
@@ -583,6 +586,9 @@ async def run(self, context):
         ep1 = context.resolveRelation(self.endpoint1)
         ep2 = context.resolveRelation(self.endpoint2)
 
+        # NB: this should really be handled by the controller when generating the
+        # bundle change plan, and this short-term workaround may be missing some
+        # aspects of the logic which the CLI client contains to handle edge cases.
         existing = [rel for rel in context.model.relations if rel.matches(ep1, ep2)]
         if existing:
             log.info('Skipping %s <-> %s; already related', ep1, ep2)