[receiver/jaeger] mark featuregates as stable

[codeboten]

This allows us to remove the deprecated modules. Pinging @frzifus as the original author

[frzifus]

Does it make sense to wait? Since the featuregate never worked.

[codeboten]

@frzifus it does, i meant to leave this as a draft to see if some of the other deps can be updated once this is removed

[github-actions]

This PR was marked stale due to lack of activity. It will be closed in 14 days.

[MovieStoreGuy]

Does it make sense to wait? Since the featuregate never worked.

Is there anything you wanted to wait for?

[dmitryax]

@frzifus can you please clarify why the feature gate never worked?

[frzifus]

Sure, the gate is evaluated inside the factory function. Since this function is called right at the beginning when the collector starts, the command line arguments are not parsed. Therefore its always "disabled".

It should be fixed once open-telemetry/opentelemetry-collector#8478 is merged.

cc @dmitryax

[codeboten]

@frzifus now that 0.89.0 is out and the feature gate issue has been addressed, should this PR be revived for the next release?

[frzifus]

sure, since it does not work for so long, it should be removed in 0.91.0, right?

[codeboten]

with v0.90.0 out, this should be good to go now. @frzifus @dmitryax

[frzifus]

Yes, thx!

[atoulme]

panic: no removal version set for Stable gate "receiver.jaegerreceiver.replaceThriftWithProto"

[dmitryax]

🎉