feat: Add custom project name option

[yamaceay]

Description of your changes

Fixes #325 by introducing a new project name field in project API, which is set optional and modifies the naming logic so that the metadata name is used as a fallback if project name not set.

I have:

• Read and followed Crossplane's contribution process.
• Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

After running the provider successfully, we can run in another terminal kubectl apply -f examples/projects/project.yaml to create a new project. The reconciliation of project works as before, successfully updating and deleting. To see the results, we can authenticate as described in hack/local-argocd-setup.sh. The following command lists the projects successfully:

> argocd proj list

NAME                  DESCRIPTION                          DESTINATIONS  SOURCES  CLUSTER-RESOURCE-WHITELIST  NAMESPACE-RESOURCE-BLACKLIST  SIGNATURE-KEYS  ORPHANED-RESOURCES  DESTINATION-SERVICE-ACCOUNTS
default                                                    *,*           *        */*                         <none>                        <none>          disabled            <none>
example-project-name  Example Project with specified name  <none>        <none>   <none>                      <none>                        <none>          disabled            <none>

[maximilianbraun]

Lgtm

[maximilianbraun]

@MisterMX Wdyt?

[MisterMX]

Usually in Crossplane external name annotation is used to store IDs / names of external resources. I'm wondering why are we using metadata.name here and not the annotation? I would rather use the annotation than a custom property.

[yamaceay]

Hi @MisterMX , before this PR, I actually had tested using external annotation name as the primary name field (and metadata.name as the fallback option, done same as in this PR).

After being told that other providers usually have a name field that we can pass, I sticked to using a custom name field. What would be your preferred option?

[yamaceay]

Just for reference, here is the link for the alternative approach: #341

[MisterMX]

Replaced by #341