Add roleARN parameter to ManagerFor in resource manager factory

This commits upadtes the ManagerFor function generator in the
resource manager factory to include the roleARN. The change also
includes:
- Updated `ManagerFor` function signature
- Modified the rmId generation to incorporate `roleARN`, creating
  a unique identifier for each account/region/roleARN combination.
- Update fake resource manager to match the new function signature

Author: a-hilaly

pkg/generate/ack/runtime_test.go

@@ -96,6 +96,7 @@ func (rmf *fakeRMF) ManagerFor(
 	*session.Session,
 	ackv1alpha1.AWSAccountID,
 	ackv1alpha1.AWSRegion,
+	ackv1alpha1.AWSResourceName,
 ) (acktypes.AWSResourceManager, error) {
 	return nil, nil
 }

templates/pkg/resource/manager_factory.go.tpl

@@ -40,8 +40,12 @@ func (f *resourceManagerFactory) ManagerFor(
 	sess *session.Session,
 	id ackv1alpha1.AWSAccountID,
 	region ackv1alpha1.AWSRegion,
+	roleARN ackv1alpha1.AWSResourceName,
 ) (acktypes.AWSResourceManager, error) {
-	rmId := fmt.Sprintf("%s/%s", id, region)
+	// We use the account ID, region, and role ARN to uniquely identify a
+	// resource manager. This helps us to avoid creating multiple resource
+	// managers for the same account/region/roleARN combination.
+	rmId := fmt.Sprintf("%s/%s/%s", id, region, roleARN)
 	f.RLock()
 	rm, found := f.rmCache[rmId]
 	f.RUnlock()