Skip to content
This repository was archived by the owner on Sep 12, 2023. It is now read-only.

Feat puppet only #73

Open
wants to merge 26 commits into
base: master
Choose a base branch
from
Open

Feat puppet only #73

wants to merge 26 commits into from

Conversation

@pcfe
Copy link
Collaborator

@pcfe pcfe commented Jul 2, 2017

This adds the possibility to have a puppet-only workflow and thus decreases the time taken to do a CI run quite noticeably.

pcfe and others added 22 commits May 16, 2017 14:45
@pcfe
echo -> info when checking MODIFIED_CONTENT_FILE
5266e37
@pcfe
fold at 240 columns instead of 80
74ca304
@pcfe
copy of config.xml but with RPM part removed
e134fe6
@pcfe
added one forced puppet run before we start waiting
fafc741
@pcfe
adjusted copy of buildtestvms.sh
23df4e4
@pcfe
added off for libvirt to buildtestvms.sh
9148467
@pcfe
introduce sleep flag to puppet-done-test.sh
cb38bc5
@pcfe
adjusted copy of buildtestvms.sh
9bf8a6b
@pcfe
wait 30 seconds betwen checks, and wait after instead of before the c…
e929214 
…heck
@pcfe
as used in testing on home server, add power control
b8cfea0
@pcfe
as used in testing on home server, add power control
5bd61ed
@pcfe
doc only
1297df0 
- Satellite 6.2 has been a minimum requirement since 2016-11-23
  added that info to the docs too
- clarified Jenkins and mock setup
- added setup steps for puppet only workflow
@pcfe
removed tight polling used during testing
537782f
@pcfe
Merge branch 'master' of github.com:RedHatSatellite/soe-ci into feat-…
fc5b353 
…puppet-only
@jeichler @pcfe
replaced post build action with conditional build step (#69)
e32d57b 
tested, fixes #67 in master
thank you
@pcfe
SELinux on Jenkins server works just fine
933767e
@pcfe
add link to yEd
0ebca8f
@pcfe
puppet-only workflow makes this rev 0.4
6e28d7a
@pcfe
clarify output
a5d5a8b
@pcfe
typo fixed; >> not »
f2ce067
@pcfe
force immediate poweroff of SUT plus note RHBZ 1417976
82c6feb
@pcfe
fixup
dcdc7e4
@pcfe pcfe requested a review from nstrug July 2, 2017 20:09
@pcfe
Copy link
Collaborator Author

pcfe commented Jul 2, 2017

this fixes #65

@pcfe pcfe requested review from boogiespook and evgeni July 4, 2017 08:59
@pcfe
Copy link
Collaborator Author

pcfe commented Jul 4, 2017

adding more reviewers as I'd like more eyeballs on this FEAT

nstrug
nstrug suggested changes Sep 28, 2017
View reviewed changes
Copy link
Member

@nstrug nstrug left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please rebase so we get the info->inform changes in and resubmit.

@nstrug nstrug mentioned this pull request Sep 28, 2017
Open
@pcfe
Copy link
Collaborator Author

pcfe commented Nov 2, 2017

note to self: rebase after #99 gets merged (or NACKed).

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

Reviewers

@evgeni evgeni Awaiting requested review from evgeni

@boogiespook boogiespook Awaiting requested review from boogiespook

1 more reviewer

@nstrug nstrug nstrug requested changes

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@pcfe @nstrug @jeichler