Commit graph

68 commits

Author SHA1 Message Date
Patrick Uiterwijk
328a30b31f The notfound is on stderr
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-05-31 03:42:41 +02:00
Patrick Uiterwijk
5b626b20a0 Fix openshift/secret-file role
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-05-31 03:40:33 +02:00
Patrick Uiterwijk
9ea8cd8b2b Attempt delete/add on change
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-05-15 21:37:27 +02:00
Patrick Uiterwijk
7deedf5d7e Actually fail the secret-file role if another one already existed for now
This role needs to be fixed to actually apply changes, so whomever sees this
gets the fun of fixing this.

Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-05-15 21:34:07 +02:00
Patrick Uiterwijk
51769d8533 Change when
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-05-03 13:59:10 +02:00
Patrick Uiterwijk
09a12cf4b5 When we try to apply project.yml, the namespace does not yet exist
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-05-03 13:58:06 +02:00
Ralph Bean
a0314660d8 Typofix. 2018-04-20 16:17:03 +00:00
Patrick Uiterwijk
312c9f8cf6 Only do secret apply if changed
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-04-19 17:37:08 +02:00
Patrick Uiterwijk
633c64b7d2 Only oc apply if the file changed
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-04-19 17:35:26 +02:00
Patrick Uiterwijk
1a0920ceee Just don't add whitespace. You don't remove waht you don't add
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-04-19 13:48:40 +00:00
Patrick Uiterwijk
671f033999 Again, this is an imagestream, not a route
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-04-19 13:48:02 +00:00
Patrick Uiterwijk
cb69ca6321 Try removing whitespace
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-04-19 13:47:03 +00:00
Patrick Uiterwijk
d32462f4c3 Fix copy-paste errors
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-04-19 15:38:43 +02:00
Patrick Uiterwijk
69303f6ee6 Remove copy-paste error
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-04-17 20:22:26 +02:00
Patrick Uiterwijk
ce449ad60c Move bodhi to opinionated imagestream role
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-04-17 20:21:47 +02:00
Patrick Uiterwijk
5dbc18df8d Make opinionated openshift/route and make bodhi use it
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-04-17 20:12:15 +02:00
Patrick Uiterwijk
8421005db1 Fix dest= missing
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-04-17 01:04:01 +00:00
Patrick Uiterwijk
ff117118a5 Use consistent, permanent filenames
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-04-17 01:02:21 +00:00
Patrick Uiterwijk
36b7948be8 Rather than run_once, use first os-master entry
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-04-17 02:38:31 +02:00
Patrick Uiterwijk
e708100dd0 Don't delete the keytab for now
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-04-16 23:39:25 +00:00
Patrick Uiterwijk
7c95ba3c45 Read the docs, Patrick
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-04-16 23:32:24 +00:00
Patrick Uiterwijk
cb61c9293f Fix include_role
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-04-16 23:31:33 +00:00
Patrick Uiterwijk
f81f5a943f Add openshift/keytab
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-04-17 01:22:18 +02:00
Patrick Uiterwijk
8de476d74d 'name' is no longer valid
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-04-16 23:49:13 +02:00
Patrick Uiterwijk
2ef2b46a37 Openshift build logs have moved to another namespace. Allow that
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-04-16 21:26:28 +00:00
Patrick Uiterwijk
3e8aaae8cf Do not conflict with ansible names
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-04-13 16:07:41 +00:00
Patrick Uiterwijk
51f9ed1a9c Turns out that that did not work
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-04-13 13:25:40 +00:00
Patrick Uiterwijk
daceb4ea24 Double <> tripple
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-04-13 13:16:34 +00:00
Patrick Uiterwijk
300ec051a3 Try double quotes
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-04-13 13:16:09 +00:00
Patrick Uiterwijk
bf33f4c83f Default to no values
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-04-13 13:14:43 +00:00
Pierre-Yves Chibon
3d057ffcd6 Finish reverting filename -> file as it was
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2018-04-12 23:42:14 +02:00
Pierre-Yves Chibon
9aa7b62166 Revert the change to openshift/object to use file as originally
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2018-04-12 23:40:49 +02:00
Patrick Uiterwijk
565b18e27e Rename file to filename
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-04-11 21:10:07 +00:00
Patrick Uiterwijk
692ddc2f78 Some objects got promoted in kubernetes 1.8 to core
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-03-09 23:29:57 +01:00
Patrick Uiterwijk
78ff12f828 Update openshift role to use namespace-local roles
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-03-09 22:53:23 +01:00
Patrick Uiterwijk
8dcf3d2faa If the secret already exists, don't crash
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-01-17 16:52:21 +00:00
Patrick Uiterwijk
b3ae5a8957 This is a 'create' on 'pods/attach', not 'attach' on 'pods'
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2017-12-15 17:15:23 +00:00
Patrick Uiterwijk
984d230e7a Allow appowners to attach to pods (Fixes #6548)
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2017-12-15 15:39:00 +00:00
Patrick Uiterwijk
b188cef81b Turns out that the subjects: thing is just informational
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2017-11-01 00:16:39 +00:00
Patrick Uiterwijk
363a554afb Allow openshift appowners in staging access to exec pods
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2017-10-23 16:03:43 +00:00
Ralph Bean
9c7390a5a4 Allow using files from the private repo in openshift secrets. 2017-10-19 00:57:45 +00:00
Patrick Uiterwijk
c591f490b8 Make appowners in staging more powerful
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2017-10-13 00:04:08 +00:00
Patrick Uiterwijk
b1f0cd0a55 Make this rerunnable
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2017-10-09 00:58:34 +00:00
Patrick Uiterwijk
de19d64c1c Add link to upstream bug for record
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2017-10-08 20:50:03 +00:00
Patrick Uiterwijk
cba7d519d4 Silly me, ClusterRoles are separate objects
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2017-10-08 20:45:48 +00:00
Patrick Uiterwijk
0d614913d1 Turns out 'global' is not a valid variable name
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2017-10-08 20:43:45 +00:00
Patrick Uiterwijk
e0f9332d86 Turns out that namespace-local roles are broken pre openshift 3.6
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2017-10-08 20:42:44 +00:00
Patrick Uiterwijk
8347455e74 I thought I learned last week that roles need their namespace specified...
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2017-10-08 19:57:59 +00:00
Patrick Uiterwijk
1ad53acd23 And *this* is plural. Man, singular and plural are hard
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2017-10-08 19:57:04 +00:00
Patrick Uiterwijk
e93950fc08 I named this singular
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2017-10-08 19:56:14 +00:00