Remove the now-unnecessary job_alloc_info_response_msg_t structure pointer

from the step context, and the assiciated allocation lookup rpc.

Creating a step context now requires only a single RPC to the controller.

src/api/step_ctx.c

@@ -69,14 +69,9 @@ slurm_step_ctx_create (job_step_create_request_msg_t *step_req)
 {
 	struct slurm_step_ctx_struct *ctx;
 	job_step_create_response_msg_t *step_resp = NULL;
-	job_alloc_info_response_msg_t *alloc_resp = NULL;
-	
-	if (slurm_allocation_lookup(step_req->job_id, &alloc_resp) < 0)
-		return NULL;
 	
 	if ((slurm_job_step_create(step_req, &step_resp) < 0) ||
 	    (step_resp == NULL)) {
-		slurm_free_job_alloc_info_response_msg(alloc_resp);
 		return NULL;	/* slurm errno already set */
 	}
 	
@@ -87,7 +82,6 @@ slurm_step_ctx_create (job_step_create_request_msg_t *step_req)
 	ctx->user_id	= step_req->user_id;
 	ctx->step_req   = _copy_step_req(step_req);
 	ctx->step_resp	= step_resp;
-	ctx->alloc_resp	= alloc_resp;
 
 	return (slurm_step_ctx)ctx;
 }
@@ -270,7 +264,6 @@ slurm_step_ctx_destroy (slurm_step_ctx ctx)
 	}
 	_free_step_req(ctx->step_req);
 	slurm_free_job_step_create_response_msg(ctx->step_resp);
-	slurm_free_job_alloc_info_response_msg(ctx->alloc_resp);
 	if (ctx->argv)
 		_xfree_char_array(&ctx->argv, ctx->argc);
 	if (ctx->env_set)

src/api/step_ctx.h

@@ -72,7 +72,6 @@ struct slurm_step_ctx_struct {
 	uint32_t job_id;	/* assigned job id */
 	uint32_t user_id;	/* user the job runs as */
 	
-	job_alloc_info_response_msg_t *alloc_resp;
 	job_step_create_request_msg_t *step_req;
 	job_step_create_response_msg_t *step_resp;
 

src/api/step_launch.c

@@ -161,7 +161,7 @@ int slurm_step_launch (slurm_step_ctx ctx,
 	_msg_thr_create(ctx->launch_state, ctx->step_req->node_count);
 
 	/* Start tasks on compute nodes */
-	launch.job_id = ctx->alloc_resp->job_id;
+	launch.job_id = ctx->step_req->job_id;
 	launch.uid = ctx->step_req->user_id;
 	launch.gid = params->gid;
 	launch.argc = params->argc;