Fixing weekly scan to run in proper controlled manner.

[mohammedzee1000]

• Removing pipelinetriggers from weekly scan
• Adding secret to store openshift master url
• Adding controller job which runs as an os cronjob.

Signed-off-by: Mohammed Zeeshan Ahmed mohammed.zee1000@gmail.com

[mohammedzee1000]

Successful run:

Creation:

$  oc process -p OS_MASTER_URL="https://os-master-1.example.com:8443"  -f configmaps/os-master-config.yaml | oc apply -f -

$  oc process -p JENKINS_SECRET_NAME=jenkins-token-ck7j8 -f weekly-scan/master-controller.yaml | oc apply -f -

If No weekly scans present:

$ docker logs 51f8d2bdab76
Logged into "https://os-master-1.lon1.centos.org:8443" as "system:serviceaccount:cccp:jenkins" using the token provided.
You have one project on this server: "cccp"
Using project "cccp".
No projects to trigger

If present:

$ docker logs 9cd913be76c1
Logged into "https://os-master-1.lon1.centos.org:8443" as "system:serviceaccount:cccp:jenkins" using the token provided.
You have one project on this server: "cccp"
Using project "cccp".
build "wscan-arrfab-nanoc-latest-1" started
....

[mohammedzee1000]

FYI you can use oc describe sa/jenkins and pick up token from there.

[bamachrn]

Even though the CI shows as running here, it actually passed. As we have updated the status context, previous context is lost.

[mohammedzee1000]

@dharmit for why separate config map directory. It is because it is a common configmap, containing dns of openshift master, which has the potential of being used elsewhere in future. Hence i brought it outside

[dharmit]

@mohammedzee1000 with potential future use cases, we can't do things right now. If in future we do end up in a scenario you think we would end up in, we can change things like this then since it's just about doing mkdir; git mv. My concern is the directory structure of our repo. It's already having things everywhere.

[mohammedzee1000]

I dont get what test is failing here

[navidshaikh]

@mohammedzee1000 : We'll need to update the deployment doc addressing how/when to create these objects in weekly-scan/ folder and their order as well.

[mohammedzee1000]

Yes, that will be needed

[navidshaikh]

I am dismissing my review and we can proceed further with @dharmit LGTM on it. Dharmit - once the CI is green, please go ahead and merge it.

[bamachrn]

dotests

[navidshaikh]

dotests

[navidshaikh]

dotests

[bamachrn]

dotests

[bamachrn]

dotests-unittests