Dynamic Supervisor Fails to Start GenServer

Hello Experts,
I am just starting to play with Supervisors and I have created the Project using mix new podder --sup
I get the Following error when I try to Invoke a Process using the Dynamic Supervisor.
This is my code.

I am using Elixir 1.7.3.

Applicaiton.ex

defmodule Podder.Application do
  # See https://hexdocs.pm/elixir/Application.html
  # for more information on OTP Applications
  @moduledoc false
  use Application

  def start(_type, _args) do
    # List all child processes to be supervised
    children = [
      # Starts a worker by calling: Podder.Worker.start_link(arg)
      # {Podder.Worker, arg},
      %{
        id: Podder.DynamicSupervisor,
        start: {Podder.DynamicSupervisor, :start_link, []}
      }
    ]

    # See https://hexdocs.pm/elixir/Supervisor.html
    # for other strategies and supported options
    opts = [strategy: :one_for_one, name: Podder.DynamicSupervisor]
    Supervisor.start_link(children, opts)
  end
end

Supervisor

defmodule Podder.DynamicSupervisor do
  use DynamicSupervisor

  def start_link() do
DynamicSupervisor.start_link(__MODULE__, :ok, name: __MODULE__)
  end

  @doc """
  Server Side
  """
  def init(_arg) do
DynamicSupervisor.init(strategy: :one_for_one)
  end

  def start_work do
{:ok, pid} = start_podcast_server
  end

  defp start_podcast_server do
spec = %{id: Podder, start: {Podder, :start_link, []}}
DynamicSupervisor.start_child(__MODULE__, spec)
  end
end

GenServer

defmodule Podder do
  use GenServer

  @moduledoc """
  Documentation for Podder.
  """

  @doc """
  Hello world.

  ## Examples

      iex> Podder.hello()
      :world

  """
  def init(state) do
    {:ok, state}
  end

  def handle_call(:fetch, _from, state) do
    new_state = Map.put(state, "fetched", 1)
    {:reply, new_state, new_state}
  end

  def handle_call(:get_state, _from, state) do
    {:reply, state, state}
  end

  @doc """
  Client Implementation
  """
  def start_link(state) do
    GenServer.start_link(__MODULE__, state)
  end

  def fetch_episodes(pid) do
    GenServer.call(pid, :fetch)
  end

  def get_episodes(pid) do
    GenServer.call(pid, :get_state)
  end
end

I am trying this from iex

iex(4)> Podder.DynamicSupervisor.start_work
** (MatchError) no match of right hand side value: {:error, {:invalid_child_spec, {{Podder, :start_link, []}, :permanent, 5000, :worker, [Podder]}}}
    (podder) lib/podcastsupervisor.ex:16: Podder.DynamicSupervisor.start_work/0
iex(4)>

The error that you’re experiencing is because in Podder.DynamicSupervisor/start_podcast_server you have defined the child spec as %{id: Podder, start: {Podder, :start_link, []}}, which means that when the dynamic supervisor tries to start the child, it will invoke Podder.start_link/0 with no arguments, but in your Podder module you have no such definition, instead you have Podder.start_link/1.

This can be fixed by either defining Podder.start_link/0 or by passing a param to the child spec in Podder.DynamicSupervisor/start_podcast_server (i.e. %{id: Podder, start: {Podder, :start_link, [:ok]}})

Sorry that was my mistake.
But actually I am do passing the args , in the updated code as so.

  def start_work do
    {:ok, pid} = start_podcast_server
  end

  defp start_podcast_server do
    spec = %{id: Podder, start: {Podder, :start_link, [%{}]}}
    DynamicSupervisor.start_child(__MODULE__, spec)
  end

But I still get the same error running in iex

 iex(5)> Podder.DynamicSupervisor.start_work
** (MatchError) no match of right hand side value: {:error, {:invalid_child_spec, {{Podder, :start_link, [%{}]}, :permanent, 5000, :worker, [Podder]}
}}
    (podder) lib/podcastsupervisor.ex:16: Podder.DynamicSupervisor.start_work/0
iex(5)> 

I tried replicating your code in a new mix project, may I ask how are you starting iex?

I tried running iex -S mix and the application is not even able to start because the current code is registering two processes with the same name (Podder.DynamicSupervisor), one for the top level supervisor and another for the actual DynamicSupervisor:

def start(_type, _args) do
    
    children = [
      %{
        id: Podder.DynamicSupervisor, #Here
        start: {Podder.DynamicSupervisor, :start_link, []}
      }
    ]

    opts = [strategy: :one_for_one, name: Podder.DynamicSupervisor] #Here
    Supervisor.start_link(children, opts)
  end

The name for the top level supervisor is normally omitted, unless you actually need it:

opts = [strategy: :one_for_one] #Here
Supervisor.start_link(children, opts)

With those changes I was able to succesfully call Podder.DynamicSupervisor.start_work

3 Likes

Have you tried

  defp start_podcast_server do
    spec = DynamicSupervisor.child_spec({Podder, %{}}, id: Podder)
    DynamicSupervisor.start_child(__MODULE__, spec)
  end

Typically, I think providing a child spec “by hand” is asking for trouble. If you’re interested, I’ve covered child specs and dynamic supervisors in more depth at http://davidsulc.com/blog/2018/07/10/pooltoy-a-toy-process-pool-manager-in-elixir-1-6-part-1-9/

1 Like

Yeah , I think I understand now what i was doing wrong.
So basically Supervisor.start_link starts the main process along with the Child Processes in my case the Dynamic Supervisor.
I was duplicating the process name and I think that probably did not even start the child process.
Even i got the error, but was unable to wrap my head around what it is.

Its a lot clearer for me now.
Thanks for the help!