Paths

Table of Contentst

Avoid to mark a site initialized on failure
ClosedPublic
Actions

Authored by dereckson on Apr 14 2023, 21:11.

Tags

None

Referenced Files

	F33604880: D3015.diff
	Fri, Jun 19, 19:38

	F33578497: D3015.diff
	Fri, Jun 19, 12:22

	F33567297: D3015.id7696.diff
	Fri, Jun 19, 08:48

	Unknown Object (File)
	Thu, Jun 18, 16:23

	Unknown Object (File)
	Thu, Jun 18, 08:38

	Unknown Object (File)
	Thu, Jun 18, 08:07

	Unknown Object (File)
	Wed, Jun 17, 16:36

	Unknown Object (File)
	Wed, Jun 17, 12:31

View All Files

Subscribers

None

Details

Reviewers

dereckson

Commits

rALK3124e5a90b11: Avoid to mark a site initialized on failure

Summary

Check the recipe status to be successful, and if not, don't mark
as initialized.

Init script should be documented to use the following exit codes:

Success: 0
Failure: 2

Test Plan

alkane init

Diff Detail

Repository

rALK Alkane

Lint

No Lint Coverage

Unit

No Test Coverage

Branch

dont-initialize-too-soon

Build Status

Buildable 4765
Build 5046: arc lint + arc unit

Event Timeline

dereckson requested review of this revision.Apr 14 2023, 21:11

dereckson created this revision.

Harbormaster completed remote builds in B4765: Diff 7695.Apr 14 2023, 21:11

dereckson accepted this revision.Apr 14 2023, 21:12

This revision is now accepted and ready to land.Apr 14 2023, 21:12

Closed by commit rALK3124e5a90b11: Avoid to mark a site initialized on failure (authored by dereckson). · Explain WhyApr 14 2023, 21:12

This revision was automatically updated to reflect the committed changes.

dereckson added a commit: rALK3124e5a90b11: Avoid to mark a site initialized on failure.

Revision Contents
Changeset List

Path

Size

src/

actions.rs

2 lines

runner/

mod.rs

2 lines

Diff 7695

View Options

src/actions.rs