Skip to content

Commit 6d82a45

Browse files
sosiouxmesosiouxme
committed
diagnostics: introduce AppCreate
1 parent 09fd06d commit 6d82a45

File tree

23 files changed

+1312
-38
lines changed

23 files changed

+1312
-38
lines changed

‎contrib/completions/bash/oc‎

Lines changed: 133 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2424,6 +2424,42 @@ _oc_adm_diagnostics_all()
24242424
flags_with_completion=()
24252425
flags_completion=()
24262426

2427+
flags+=("--appcreate-admission-timeout=")
2428+
local_nonpersistent_flags+=("--appcreate-admission-timeout=")
2429+
flags+=("--appcreate-app-image=")
2430+
local_nonpersistent_flags+=("--appcreate-app-image=")
2431+
flags+=("--appcreate-app-name=")
2432+
local_nonpersistent_flags+=("--appcreate-app-name=")
2433+
flags+=("--appcreate-app-port=")
2434+
local_nonpersistent_flags+=("--appcreate-app-port=")
2435+
flags+=("--appcreate-deploy-timeout=")
2436+
local_nonpersistent_flags+=("--appcreate-deploy-timeout=")
2437+
flags+=("--appcreate-http-retries=")
2438+
local_nonpersistent_flags+=("--appcreate-http-retries=")
2439+
flags+=("--appcreate-http-timeout=")
2440+
local_nonpersistent_flags+=("--appcreate-http-timeout=")
2441+
flags+=("--appcreate-keep-app")
2442+
local_nonpersistent_flags+=("--appcreate-keep-app")
2443+
flags+=("--appcreate-keep-project")
2444+
local_nonpersistent_flags+=("--appcreate-keep-project")
2445+
flags+=("--appcreate-node-selector=")
2446+
local_nonpersistent_flags+=("--appcreate-node-selector=")
2447+
flags+=("--appcreate-project=")
2448+
local_nonpersistent_flags+=("--appcreate-project=")
2449+
flags+=("--appcreate-project-base=")
2450+
local_nonpersistent_flags+=("--appcreate-project-base=")
2451+
flags+=("--appcreate-result-dir=")
2452+
local_nonpersistent_flags+=("--appcreate-result-dir=")
2453+
flags+=("--appcreate-route-host=")
2454+
local_nonpersistent_flags+=("--appcreate-route-host=")
2455+
flags+=("--appcreate-route-port=")
2456+
local_nonpersistent_flags+=("--appcreate-route-port=")
2457+
flags+=("--appcreate-skip-route-connect")
2458+
local_nonpersistent_flags+=("--appcreate-skip-route-connect")
2459+
flags+=("--appcreate-skip-route-test")
2460+
local_nonpersistent_flags+=("--appcreate-skip-route-test")
2461+
flags+=("--appcreate-skip-service-connect")
2462+
local_nonpersistent_flags+=("--appcreate-skip-service-connect")
24272463
flags+=("--cluster-context=")
24282464
local_nonpersistent_flags+=("--cluster-context=")
24292465
flags+=("--config=")
@@ -2555,6 +2591,102 @@ _oc_adm_diagnostics_analyzelogs()
25552591
noun_aliases=()
25562592
}
25572593

2594+
_oc_adm_diagnostics_appcreate()
2595+
{
2596+
last_command="oc_adm_diagnostics_appcreate"
2597+
commands=()
2598+
2599+
flags=()
2600+
two_word_flags=()
2601+
local_nonpersistent_flags=()
2602+
flags_with_completion=()
2603+
flags_completion=()
2604+
2605+
flags+=("--admission-timeout=")
2606+
local_nonpersistent_flags+=("--admission-timeout=")
2607+
flags+=("--app-image=")
2608+
local_nonpersistent_flags+=("--app-image=")
2609+
flags+=("--app-name=")
2610+
local_nonpersistent_flags+=("--app-name=")
2611+
flags+=("--app-port=")
2612+
local_nonpersistent_flags+=("--app-port=")
2613+
flags+=("--cluster-context=")
2614+
local_nonpersistent_flags+=("--cluster-context=")
2615+
flags+=("--config=")
2616+
flags_with_completion+=("--config")
2617+
flags_completion+=("_filedir")
2618+
local_nonpersistent_flags+=("--config=")
2619+
flags+=("--context=")
2620+
local_nonpersistent_flags+=("--context=")
2621+
flags+=("--deploy-timeout=")
2622+
local_nonpersistent_flags+=("--deploy-timeout=")
2623+
flags+=("--diaglevel=")
2624+
two_word_flags+=("-l")
2625+
local_nonpersistent_flags+=("--diaglevel=")
2626+
flags+=("--http-retries=")
2627+
local_nonpersistent_flags+=("--http-retries=")
2628+
flags+=("--http-timeout=")
2629+
local_nonpersistent_flags+=("--http-timeout=")
2630+
flags+=("--keep-app")
2631+
local_nonpersistent_flags+=("--keep-app")
2632+
flags+=("--keep-project")
2633+
local_nonpersistent_flags+=("--keep-project")
2634+
flags+=("--loglevel=")
2635+
local_nonpersistent_flags+=("--loglevel=")
2636+
flags+=("--logspec=")
2637+
local_nonpersistent_flags+=("--logspec=")
2638+
flags+=("--node-selector=")
2639+
local_nonpersistent_flags+=("--node-selector=")
2640+
flags+=("--prevent-modification")
2641+
local_nonpersistent_flags+=("--prevent-modification")
2642+
flags+=("--project=")
2643+
local_nonpersistent_flags+=("--project=")
2644+
flags+=("--project-base=")
2645+
local_nonpersistent_flags+=("--project-base=")
2646+
flags+=("--result-dir=")
2647+
local_nonpersistent_flags+=("--result-dir=")
2648+
flags+=("--route-host=")
2649+
local_nonpersistent_flags+=("--route-host=")
2650+
flags+=("--route-port=")
2651+
local_nonpersistent_flags+=("--route-port=")
2652+
flags+=("--skip-route-connect")
2653+
local_nonpersistent_flags+=("--skip-route-connect")
2654+
flags+=("--skip-route-test")
2655+
local_nonpersistent_flags+=("--skip-route-test")
2656+
flags+=("--skip-service-connect")
2657+
local_nonpersistent_flags+=("--skip-service-connect")
2658+
flags+=("--v=")
2659+
local_nonpersistent_flags+=("--v=")
2660+
flags+=("--vmodule=")
2661+
local_nonpersistent_flags+=("--vmodule=")
2662+
flags+=("--as=")
2663+
flags+=("--as-group=")
2664+
flags+=("--cache-dir=")
2665+
flags+=("--certificate-authority=")
2666+
flags_with_completion+=("--certificate-authority")
2667+
flags_completion+=("_filedir")
2668+
flags+=("--client-certificate=")
2669+
flags_with_completion+=("--client-certificate")
2670+
flags_completion+=("_filedir")
2671+
flags+=("--client-key=")
2672+
flags_with_completion+=("--client-key")
2673+
flags_completion+=("_filedir")
2674+
flags+=("--cluster=")
2675+
flags+=("--insecure-skip-tls-verify")
2676+
flags+=("--log-flush-frequency=")
2677+
flags+=("--match-server-version")
2678+
flags+=("--namespace=")
2679+
two_word_flags+=("-n")
2680+
flags+=("--request-timeout=")
2681+
flags+=("--server=")
2682+
flags+=("--token=")
2683+
flags+=("--user=")
2684+
2685+
must_have_one_flag=()
2686+
must_have_one_noun=()
2687+
noun_aliases=()
2688+
}
2689+
25582690
_oc_adm_diagnostics_clusterregistry()
25592691
{
25602692
last_command="oc_adm_diagnostics_clusterregistry"
@@ -3550,6 +3682,7 @@ _oc_adm_diagnostics()
35503682
commands+=("aggregatedlogging")
35513683
commands+=("all")
35523684
commands+=("analyzelogs")
3685+
commands+=("appcreate")
35533686
commands+=("clusterregistry")
35543687
commands+=("clusterrolebindings")
35553688
commands+=("clusterroles")

‎contrib/completions/zsh/oc‎

Lines changed: 133 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2566,6 +2566,42 @@ _oc_adm_diagnostics_all()
25662566
flags_with_completion=()
25672567
flags_completion=()
25682568

2569+
flags+=("--appcreate-admission-timeout=")
2570+
local_nonpersistent_flags+=("--appcreate-admission-timeout=")
2571+
flags+=("--appcreate-app-image=")
2572+
local_nonpersistent_flags+=("--appcreate-app-image=")
2573+
flags+=("--appcreate-app-name=")
2574+
local_nonpersistent_flags+=("--appcreate-app-name=")
2575+
flags+=("--appcreate-app-port=")
2576+
local_nonpersistent_flags+=("--appcreate-app-port=")
2577+
flags+=("--appcreate-deploy-timeout=")
2578+
local_nonpersistent_flags+=("--appcreate-deploy-timeout=")
2579+
flags+=("--appcreate-http-retries=")
2580+
local_nonpersistent_flags+=("--appcreate-http-retries=")
2581+
flags+=("--appcreate-http-timeout=")
2582+
local_nonpersistent_flags+=("--appcreate-http-timeout=")
2583+
flags+=("--appcreate-keep-app")
2584+
local_nonpersistent_flags+=("--appcreate-keep-app")
2585+
flags+=("--appcreate-keep-project")
2586+
local_nonpersistent_flags+=("--appcreate-keep-project")
2587+
flags+=("--appcreate-node-selector=")
2588+
local_nonpersistent_flags+=("--appcreate-node-selector=")
2589+
flags+=("--appcreate-project=")
2590+
local_nonpersistent_flags+=("--appcreate-project=")
2591+
flags+=("--appcreate-project-base=")
2592+
local_nonpersistent_flags+=("--appcreate-project-base=")
2593+
flags+=("--appcreate-result-dir=")
2594+
local_nonpersistent_flags+=("--appcreate-result-dir=")
2595+
flags+=("--appcreate-route-host=")
2596+
local_nonpersistent_flags+=("--appcreate-route-host=")
2597+
flags+=("--appcreate-route-port=")
2598+
local_nonpersistent_flags+=("--appcreate-route-port=")
2599+
flags+=("--appcreate-skip-route-connect")
2600+
local_nonpersistent_flags+=("--appcreate-skip-route-connect")
2601+
flags+=("--appcreate-skip-route-test")
2602+
local_nonpersistent_flags+=("--appcreate-skip-route-test")
2603+
flags+=("--appcreate-skip-service-connect")
2604+
local_nonpersistent_flags+=("--appcreate-skip-service-connect")
25692605
flags+=("--cluster-context=")
25702606
local_nonpersistent_flags+=("--cluster-context=")
25712607
flags+=("--config=")
@@ -2697,6 +2733,102 @@ _oc_adm_diagnostics_analyzelogs()
26972733
noun_aliases=()
26982734
}
26992735

2736+
_oc_adm_diagnostics_appcreate()
2737+
{
2738+
last_command="oc_adm_diagnostics_appcreate"
2739+
commands=()
2740+
2741+
flags=()
2742+
two_word_flags=()
2743+
local_nonpersistent_flags=()
2744+
flags_with_completion=()
2745+
flags_completion=()
2746+
2747+
flags+=("--admission-timeout=")
2748+
local_nonpersistent_flags+=("--admission-timeout=")
2749+
flags+=("--app-image=")
2750+
local_nonpersistent_flags+=("--app-image=")
2751+
flags+=("--app-name=")
2752+
local_nonpersistent_flags+=("--app-name=")
2753+
flags+=("--app-port=")
2754+
local_nonpersistent_flags+=("--app-port=")
2755+
flags+=("--cluster-context=")
2756+
local_nonpersistent_flags+=("--cluster-context=")
2757+
flags+=("--config=")
2758+
flags_with_completion+=("--config")
2759+
flags_completion+=("_filedir")
2760+
local_nonpersistent_flags+=("--config=")
2761+
flags+=("--context=")
2762+
local_nonpersistent_flags+=("--context=")
2763+
flags+=("--deploy-timeout=")
2764+
local_nonpersistent_flags+=("--deploy-timeout=")
2765+
flags+=("--diaglevel=")
2766+
two_word_flags+=("-l")
2767+
local_nonpersistent_flags+=("--diaglevel=")
2768+
flags+=("--http-retries=")
2769+
local_nonpersistent_flags+=("--http-retries=")
2770+
flags+=("--http-timeout=")
2771+
local_nonpersistent_flags+=("--http-timeout=")
2772+
flags+=("--keep-app")
2773+
local_nonpersistent_flags+=("--keep-app")
2774+
flags+=("--keep-project")
2775+
local_nonpersistent_flags+=("--keep-project")
2776+
flags+=("--loglevel=")
2777+
local_nonpersistent_flags+=("--loglevel=")
2778+
flags+=("--logspec=")
2779+
local_nonpersistent_flags+=("--logspec=")
2780+
flags+=("--node-selector=")
2781+
local_nonpersistent_flags+=("--node-selector=")
2782+
flags+=("--prevent-modification")
2783+
local_nonpersistent_flags+=("--prevent-modification")
2784+
flags+=("--project=")
2785+
local_nonpersistent_flags+=("--project=")
2786+
flags+=("--project-base=")
2787+
local_nonpersistent_flags+=("--project-base=")
2788+
flags+=("--result-dir=")
2789+
local_nonpersistent_flags+=("--result-dir=")
2790+
flags+=("--route-host=")
2791+
local_nonpersistent_flags+=("--route-host=")
2792+
flags+=("--route-port=")
2793+
local_nonpersistent_flags+=("--route-port=")
2794+
flags+=("--skip-route-connect")
2795+
local_nonpersistent_flags+=("--skip-route-connect")
2796+
flags+=("--skip-route-test")
2797+
local_nonpersistent_flags+=("--skip-route-test")
2798+
flags+=("--skip-service-connect")
2799+
local_nonpersistent_flags+=("--skip-service-connect")
2800+
flags+=("--v=")
2801+
local_nonpersistent_flags+=("--v=")
2802+
flags+=("--vmodule=")
2803+
local_nonpersistent_flags+=("--vmodule=")
2804+
flags+=("--as=")
2805+
flags+=("--as-group=")
2806+
flags+=("--cache-dir=")
2807+
flags+=("--certificate-authority=")
2808+
flags_with_completion+=("--certificate-authority")
2809+
flags_completion+=("_filedir")
2810+
flags+=("--client-certificate=")
2811+
flags_with_completion+=("--client-certificate")
2812+
flags_completion+=("_filedir")
2813+
flags+=("--client-key=")
2814+
flags_with_completion+=("--client-key")
2815+
flags_completion+=("_filedir")
2816+
flags+=("--cluster=")
2817+
flags+=("--insecure-skip-tls-verify")
2818+
flags+=("--log-flush-frequency=")
2819+
flags+=("--match-server-version")
2820+
flags+=("--namespace=")
2821+
two_word_flags+=("-n")
2822+
flags+=("--request-timeout=")
2823+
flags+=("--server=")
2824+
flags+=("--token=")
2825+
flags+=("--user=")
2826+
2827+
must_have_one_flag=()
2828+
must_have_one_noun=()
2829+
noun_aliases=()
2830+
}
2831+
27002832
_oc_adm_diagnostics_clusterregistry()
27012833
{
27022834
last_command="oc_adm_diagnostics_clusterregistry"
@@ -3692,6 +3824,7 @@ _oc_adm_diagnostics()
36923824
commands+=("aggregatedlogging")
36933825
commands+=("all")
36943826
commands+=("analyzelogs")
3827+
commands+=("appcreate")
36953828
commands+=("clusterregistry")
36963829
commands+=("clusterrolebindings")
36973830
commands+=("clusterroles")

‎docs/man/man1/.files_generated_oc‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@ oc-adm-create-signer-cert.1
4040
oc-adm-diagnostics-aggregatedlogging.1
4141
oc-adm-diagnostics-all.1
4242
oc-adm-diagnostics-analyzelogs.1
43+
oc-adm-diagnostics-appcreate.1
4344
oc-adm-diagnostics-clusterregistry.1
4445
oc-adm-diagnostics-clusterrolebindings.1
4546
oc-adm-diagnostics-clusterroles.1
@@ -193,6 +194,7 @@ oc-ex-config.1
193194
oc-ex-diagnostics-aggregatedlogging.1
194195
oc-ex-diagnostics-all.1
195196
oc-ex-diagnostics-analyzelogs.1
197+
oc-ex-diagnostics-appcreate.1
196198
oc-ex-diagnostics-clusterregistry.1
197199
oc-ex-diagnostics-clusterrolebindings.1
198200
oc-ex-diagnostics-clusterroles.1

‎docs/man/man1/oc-adm-diagnostics-appcreate.1‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
This file is autogenerated, but we've stopped checking such files into the
2+
repository to reduce the need for rebases. Please run hack/generate-docs.sh to
3+
populate this file.

‎docs/man/man1/oc-ex-diagnostics-appcreate.1‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
This file is autogenerated, but we've stopped checking such files into the
2+
repository to reduce the need for rebases. Please run hack/generate-docs.sh to
3+
populate this file.

‎pkg/oc/admin/diagnostics/cluster.go‎

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ import (
1717
oauthclient "github.com/openshift/origin/pkg/oauth/generated/internalclientset"
1818
clustdiags "github.com/openshift/origin/pkg/oc/admin/diagnostics/diagnostics/cluster"
1919
agldiags "github.com/openshift/origin/pkg/oc/admin/diagnostics/diagnostics/cluster/aggregated_logging"
20+
appcreate "github.com/openshift/origin/pkg/oc/admin/diagnostics/diagnostics/cluster/app_create"
2021
"github.com/openshift/origin/pkg/oc/admin/diagnostics/diagnostics/types"
2122
osclientcmd "github.com/openshift/origin/pkg/oc/cli/util/clientcmd"
2223
projectclient "github.com/openshift/origin/pkg/project/generated/internalclientset"
@@ -30,6 +31,7 @@ import (
3031
func availableClusterDiagnostics() types.DiagnosticList {
3132
return types.DiagnosticList{
3233
&agldiags.AggregatedLogging{},
34+
appcreate.NewDefaultAppCreateDiagnostic(),
3335
&clustdiags.ClusterRegistry{},
3436
&clustdiags.ClusterRouter{},
3537
&clustdiags.ClusterRoles{},
@@ -95,6 +97,16 @@ func (o DiagnosticsOptions) buildClusterDiagnostics(rawConfig *clientcmdapi.Conf
9597
switch diagnosticName {
9698
case agldiags.AggregatedLoggingName:
9799
d = agldiags.NewAggregatedLogging(o.MasterConfigLocation, kclusterClient, oauthClient.Oauth(), projectClient.Project(), routeClient.Route(), oauthorizationClient.Authorization(), appsClient.Apps(), securityClient.Security())
100+
case appcreate.AppCreateName:
101+
ac := o.ParameterizedDiagnostics[diagnosticName].(*appcreate.AppCreate)
102+
ac.KubeClient = kclusterClient
103+
ac.ProjectClient = projectClient.Project()
104+
ac.RouteClient = routeClient
105+
ac.RoleBindingClient = oauthorizationClient.Authorization()
106+
ac.SARClient = kclusterClient.Authorization()
107+
ac.AppsClient = appsClient
108+
ac.PreventModification = o.PreventModification
109+
d = ac
98110
case clustdiags.NodeDefinitionsName:
99111
d = &clustdiags.NodeDefinitions{KubeClient: kclusterClient}
100112
case clustdiags.MasterNodeName:

‎pkg/oc/admin/diagnostics/diagnostics.go‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -269,6 +269,8 @@ func bindIndividualFlags(diag types.ParameterizedDiagnostic, prefix string, flag
269269
flags.StringVar(target, name, param.Default.(string), param.Description)
270270
case *int:
271271
flags.IntVar(target, name, param.Default.(int), param.Description)
272+
case *int64:
273+
flags.Int64Var(target, name, param.Default.(int64), param.Description)
272274
case *bool:
273275
flags.BoolVar(target, name, param.Default.(bool), param.Description)
274276
default:

0 commit comments

Comments
 (0)