Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

AIAP: Shift gates to use AIAP #659

Open
eak13 opened this issue Oct 20, 2021 · 3 comments
Open

AIAP: Shift gates to use AIAP #659

eak13 opened this issue Oct 20, 2021 · 3 comments
Assignees
@michael-beaver
Labels
4-CI/CD Relates to issues with Zuul & gating enhancement New feature or request
Milestone
v2.2

Comments

@eak13
Copy link

eak13 commented Oct 20, 2021

We need to shift our upstream gates to use AIAP exclusively as the functional testing gate in our Airshipctl Zuul and Jenkins systems, so that we are using the same testing mechanism end to end. This requires #658 to be in place first before this can be implemented.
@eak13 eak13 added enhancement New feature or request 4-CI/CD Relates to issues with Zuul & gating triage Needs evaluation by project members labels Oct 20, 2021
@eak13 eak13 added this to the v2.1 milestone Oct 20, 2021
@eak13 eak13 mentioned this issue Oct 20, 2021
Closed
@eak13 eak13 modified the milestones: v2.1, v2.2 Oct 27, 2021
@eak13 eak13 removed the triage Needs evaluation by project members label Oct 27, 2021
airshipbot pushed a commit that referenced this issue Nov 11, 2021
AIAP Gate - Update script to use status-checker
44f2b82 
This change updates the 20_apply_aiap.sh script to utilize the
status-checker container to determine if airship-in-a-pod has
successfully completed. It also includes a check to fail when
the status-checker has detected a failure.

Relates-To: #659
Change-Id: I3673de7157f498778154c798878ab20245396c62
airshipbot pushed a commit that referenced this issue Dec 3, 2021
AIAP - Extract logs from runner container
56972b4 
This adds a new script to the runner container that is run in the
entrypoint which runs the log gathering ansible playbook from
inside the runner container. The gate then extracts these logs
with a new role.

This also updates the image_build script to fix how it changes
the container imagePullPolicy

Closes: #658
Relates-To: #659
Change-Id: I24d11c66e7b71852256e164343f7bb2f331d1fef
@sirajyasin
Copy link
Contributor

I can see the issue #658 is now completed. I can test log extraction feature and enable voting for aiap zuul gate. can someone assign this issue to me ?

@michael-beaver
Copy link

@sirajyasin Sorry I didn't get this assigned to me, I am already working this as well as #658

@joshuaherrera
Copy link
Contributor

joshuaherrera commented Jan 19, 2022 •
edited
Loading

This related patchset https://review.opendev.org/c/airship/airshipctl/+/824738
is needed so that the /var/local-path-provisioner exists for the persistent volume and persistent volume claim.

The Zuul job was timing out because this directory was not found. With this PS, the AIAP Zuul job no longer times out, but it seems to still fail, so the failures would need to be resolved before shifting the gates over to AIAP.

I was using this test PS to log output from the Zuul job (https://review.opendev.org/c/airship/airshipctl/+/823727)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@michael-beaver michael-beaver

Labels
4-CI/CD Relates to issues with Zuul & gating enhancement New feature or request
Projects
None yet
Milestone
v2.2
Development

No branches or pull requests
4 participants
@eak13 @michael-beaver @joshuaherrera @sirajyasin