[demo] Remove FeatureFlagService and add flagd

[julianocosta89]

The PR open-telemetry/opentelemetry-demo#1388 removed FeatureFlag Service and replaced it with flagd.

This PR updates the charts to reflect the latest changes on the demo.
It also updates the dependencies charts.

[julianocosta89]

@puckpuck you mention in the .env file that # must also update version field in /src/grafana/provisioning/datasources/opensearch.yml, does that also apply for the Helm Chart?

I've updated opensearch, but I couldn't find where to update it on Grafana datasource.

[puckpuck]

Line 809 of the values file for the Helm chart.

I'm on PTO using my phone so apologies for the short message.

[julianocosta89]

Line 809 of the values file for the Helm chart.

Thank you!

[julianocosta89]

Hey @TylerHelmuth 👋🏽

Do you happen to know what am I doing wrong here?
I'm trying to create the flagd configMap and attach it to the container, but I'm failing miserably 😭

[AlexPSplunk]

For me the flagd container just runs and exits as if no config has been specified. Configmap appears to be mounted at /etc/flagd. Below is the output I get when looking at the flagd container logs.
Is it possible that the flagd container build is bad?

$ kubectl logs astroshop-edu-o11y-flagd-6f85c496b9-j2f5f

		 ______   __       ________   _______    ______
		/_____/\ /_/\     /_______/\ /______/\  /_____/\
		\::::_\/_\:\ \    \::: _  \ \\::::__\/__\:::_ \ \
		 \:\/___/\\:\ \    \::(_)  \ \\:\ /____/\\:\ \ \ \
		  \:::._\/ \:\ \____\:: __  \ \\:\\_  _\/ \:\ \ \ \
		   \:\ \    \:\/___/\\:.\ \  \ \\:\_\ \ \  \:\/.:| |
		    \_\/     \_____\/ \__\/\__\/ \_____\/   \____/_/

Flagd is a simple command line tool for fetching and presenting feature flags to services. It is designed to conform to Open Feature schema for flag definitions.

Usage:
  flagd [command]

Available Commands:
  completion  Generate the autocompletion script for the specified shell
  help        Help about any command
  start       Start flagd
  version     Print the version number of flagd

Flags:
      --config string   config file (default is $HOME/.agent.yaml)
  -x, --debug           verbose logging
  -h, --help            help for flagd

Use "flagd [command] --help" for more information about a command.```

[puckpuck]

@julianocosta89 I pushed a commit to your branch to get flagd working. There were a few small other issues, but the main one was having to specify the command arguments.

This did bring up an interesting point. Right now we have 2 environment variables, FLAGD_HOST and FLAGD_PORT. The demo uses both of these, but for everything else, we use a single _ADDR variant instead. I think we should update the demo to use this single variable instead.

[julianocosta89]

@puckpuck I actually started doing that, but glad doesn't support it.

https://flagd.dev/reference/specifications/in-process-providers/?h=environmen#explicit-declaration

I'm on the phone so the link may not be pointing to the right place, but check out the environment variables in this page.

[julianocosta89]

BTW, thanks for taking a look and fixing the PR

[julianocosta89]

Would we be able to get this merged?
We are getting a bunch of PRs in the demo that would require helm updates, but the flagd is not merged yet.

[puckpuck]

We need the demo to cut a release first, then we can target those images in this Helm chart to ensure all CI tests pass.

[julianocosta89]

@puckpuck this one should be good to go! 🥳

[puckpuck]

@julianocosta89 I updated NOTES.txt to remove references to the old feature flag UI, and replaced demo.flagd.json with the same from the demo repo to ensure they are the same.

Everything looks good!