InformerEventSource should have both SecondaryToPrimary and PrimaryToSecondary mappers

[Kate Stanley]

According to the Java operator SDK docs, if the mapping between a primary and secondary resource is not 1-1 then when the event source is created it needs to have both a SecondaryToPrimary and PrimaryToSecondary mapper.

Since more than one different KafkaAccess CR could reference the same Kafka CR, for all of the resources that are related to the Kafka CR (e.g. the Kafka CR, KafkaUser CR, Secrets etc), when we define the event sources we need both types of mapper. Currently for most we only provider a SecondaryToPrimary mapper.

We should add PrimaryToSecondary mappers for the kafkaEventSource and strimziSecretEventSource.

References:

• Java Operator SDK docs
• prepareEventSources function
• Example mapper for KafkaUser

[Kate Stanley]

Update: This is only required if the function context.getSecondaryResource is called, otherwise it is not required. This means with the current operator this is only required for the Kafka Access secret event source. However if we upgrade to the latest JOSDK version the secret can be handled as a dependent resource and we will likely not need it at all.

[Kate Stanley]

As part of #30 we added a PrimaryToSecondary mapper for Kafka. Since for all other resources we are using the event source directly I do not think we need to create any other PrimaryToSecondary mappers at the moment.

Closing this issue and we can open a new one if this changes.