Check the value you’re using for $post_id when calling media_sideload_image(). According to the docs the function is supposed to attach the file. If it’s not the only thing I can think of is that the value your using there is somehow incorrect. It must be an integer for an existing post.
Welcome to the Advanced Custom Fields community forum.
Browse through ideas, snippets of code, questions and answers between fellow ACF users