Move pudl_etl_job_factory back to pudl.etl.cli.py

[bendnorman]

Overview

Closes #3708.

What problem does this address?
#3687 broke the pudl_etl command because build_reconstructable_job needs pudl_etl_job_factory to be in the same file:

pudl/src/pudl/etl/cli.py

Lines 131 to 139 in c153929

	pudl_etl_reconstructable_job = build_reconstructable_job(
	"pudl.etl.cli",
	"pudl_etl_job_factory",
	reconstructable_kwargs={
	"loglevel": loglevel,
	"logfile": logfile,
	"process_epacems": process_epacems,
	},
	)

What did you change?
This PR just reverts the pudl_etl_job_factory move that happened in #3687.

I also fleshed out this issue which would allow us to migrate away from the experimental and confusing build_reconstructable_job method.

Testing

How did you make sure this worked? How can a reviewer verify this?

• I ran make pytest-coverage and pudl_etl and they both started without the error.

To-do list

Preview Give feedback

1. If updating analyses or data processing functions: make sure to update or write data validation tests (e.g., test_minmax_rows())
2. Update the release notes: reference the PR and related issues.
3. Ensure docs build, unit & integration tests, and test coverage pass locally with make pytest-coverage (otherwise the merge queue may reject your PR)
4. Review the PR yourself and call out any questions or issues you have
5. For minor ETL changes or data additions, once make pytest-coverage passes, make sure you have a fresh full PUDL DB downloaded locally, materialize new/changed assets and all their downstream assets and run relevant data validation tests using pytest and --live-dbs.
6. For significant ETL, data coverage or analysis changes, once make pytest-coverage passes, ensure the full ETL runs locally and run data validation tests using make pytest-validate (a ~10 hour run). If you can't run this locally, run the build-deploy-pudl GitHub Action (or ask someone with permissions to). Then, check the logs on the #pudl-deployments Slack channel or gs://builds.catalyst.coop.