Prioritize StorageOperationFailedCondition
Prioritize StorageOperationFailedCondition over other artifact outdated and unavailable conditions so that when artifact is failing due to storage operation, it's visble in the ready status condition, making the reason for not ready more accurate. Signed-off-by: Sunny <darkowlzz@protonmail.com>
This commit is contained in:
parent
a3cbe6ee46
commit
a102d95cf9
|
@ -75,23 +75,23 @@ var bucketReadyCondition = summarize.Conditions{
|
||||||
Target: meta.ReadyCondition,
|
Target: meta.ReadyCondition,
|
||||||
Owned: []string{
|
Owned: []string{
|
||||||
sourcev1.FetchFailedCondition,
|
sourcev1.FetchFailedCondition,
|
||||||
sourcev1.ArtifactOutdatedCondition,
|
|
||||||
sourcev1.StorageOperationFailedCondition,
|
sourcev1.StorageOperationFailedCondition,
|
||||||
|
sourcev1.ArtifactOutdatedCondition,
|
||||||
meta.ReadyCondition,
|
meta.ReadyCondition,
|
||||||
meta.ReconcilingCondition,
|
meta.ReconcilingCondition,
|
||||||
meta.StalledCondition,
|
meta.StalledCondition,
|
||||||
},
|
},
|
||||||
Summarize: []string{
|
Summarize: []string{
|
||||||
sourcev1.FetchFailedCondition,
|
sourcev1.FetchFailedCondition,
|
||||||
sourcev1.ArtifactOutdatedCondition,
|
|
||||||
sourcev1.StorageOperationFailedCondition,
|
sourcev1.StorageOperationFailedCondition,
|
||||||
|
sourcev1.ArtifactOutdatedCondition,
|
||||||
meta.StalledCondition,
|
meta.StalledCondition,
|
||||||
meta.ReconcilingCondition,
|
meta.ReconcilingCondition,
|
||||||
},
|
},
|
||||||
NegativePolarity: []string{
|
NegativePolarity: []string{
|
||||||
sourcev1.FetchFailedCondition,
|
sourcev1.FetchFailedCondition,
|
||||||
sourcev1.ArtifactOutdatedCondition,
|
|
||||||
sourcev1.StorageOperationFailedCondition,
|
sourcev1.StorageOperationFailedCondition,
|
||||||
|
sourcev1.ArtifactOutdatedCondition,
|
||||||
meta.StalledCondition,
|
meta.StalledCondition,
|
||||||
meta.ReconcilingCondition,
|
meta.ReconcilingCondition,
|
||||||
},
|
},
|
||||||
|
|
|
@ -62,9 +62,9 @@ var gitRepositoryReadyCondition = summarize.Conditions{
|
||||||
Owned: []string{
|
Owned: []string{
|
||||||
sourcev1.SourceVerifiedCondition,
|
sourcev1.SourceVerifiedCondition,
|
||||||
sourcev1.FetchFailedCondition,
|
sourcev1.FetchFailedCondition,
|
||||||
|
sourcev1.StorageOperationFailedCondition,
|
||||||
sourcev1.IncludeUnavailableCondition,
|
sourcev1.IncludeUnavailableCondition,
|
||||||
sourcev1.ArtifactOutdatedCondition,
|
sourcev1.ArtifactOutdatedCondition,
|
||||||
sourcev1.StorageOperationFailedCondition,
|
|
||||||
meta.ReadyCondition,
|
meta.ReadyCondition,
|
||||||
meta.ReconcilingCondition,
|
meta.ReconcilingCondition,
|
||||||
meta.StalledCondition,
|
meta.StalledCondition,
|
||||||
|
@ -73,16 +73,16 @@ var gitRepositoryReadyCondition = summarize.Conditions{
|
||||||
sourcev1.IncludeUnavailableCondition,
|
sourcev1.IncludeUnavailableCondition,
|
||||||
sourcev1.SourceVerifiedCondition,
|
sourcev1.SourceVerifiedCondition,
|
||||||
sourcev1.FetchFailedCondition,
|
sourcev1.FetchFailedCondition,
|
||||||
sourcev1.ArtifactOutdatedCondition,
|
|
||||||
sourcev1.StorageOperationFailedCondition,
|
sourcev1.StorageOperationFailedCondition,
|
||||||
|
sourcev1.ArtifactOutdatedCondition,
|
||||||
meta.StalledCondition,
|
meta.StalledCondition,
|
||||||
meta.ReconcilingCondition,
|
meta.ReconcilingCondition,
|
||||||
},
|
},
|
||||||
NegativePolarity: []string{
|
NegativePolarity: []string{
|
||||||
sourcev1.FetchFailedCondition,
|
sourcev1.FetchFailedCondition,
|
||||||
sourcev1.IncludeUnavailableCondition,
|
sourcev1.IncludeUnavailableCondition,
|
||||||
sourcev1.ArtifactOutdatedCondition,
|
|
||||||
sourcev1.StorageOperationFailedCondition,
|
sourcev1.StorageOperationFailedCondition,
|
||||||
|
sourcev1.ArtifactOutdatedCondition,
|
||||||
meta.StalledCondition,
|
meta.StalledCondition,
|
||||||
meta.ReconcilingCondition,
|
meta.ReconcilingCondition,
|
||||||
},
|
},
|
||||||
|
|
|
@ -71,8 +71,8 @@ var helmChartReadyCondition = summarize.Conditions{
|
||||||
Owned: []string{
|
Owned: []string{
|
||||||
sourcev1.BuildFailedCondition,
|
sourcev1.BuildFailedCondition,
|
||||||
sourcev1.FetchFailedCondition,
|
sourcev1.FetchFailedCondition,
|
||||||
sourcev1.ArtifactOutdatedCondition,
|
|
||||||
sourcev1.StorageOperationFailedCondition,
|
sourcev1.StorageOperationFailedCondition,
|
||||||
|
sourcev1.ArtifactOutdatedCondition,
|
||||||
meta.ReadyCondition,
|
meta.ReadyCondition,
|
||||||
meta.ReconcilingCondition,
|
meta.ReconcilingCondition,
|
||||||
meta.StalledCondition,
|
meta.StalledCondition,
|
||||||
|
@ -80,16 +80,16 @@ var helmChartReadyCondition = summarize.Conditions{
|
||||||
Summarize: []string{
|
Summarize: []string{
|
||||||
sourcev1.BuildFailedCondition,
|
sourcev1.BuildFailedCondition,
|
||||||
sourcev1.FetchFailedCondition,
|
sourcev1.FetchFailedCondition,
|
||||||
sourcev1.ArtifactOutdatedCondition,
|
|
||||||
sourcev1.StorageOperationFailedCondition,
|
sourcev1.StorageOperationFailedCondition,
|
||||||
|
sourcev1.ArtifactOutdatedCondition,
|
||||||
meta.StalledCondition,
|
meta.StalledCondition,
|
||||||
meta.ReconcilingCondition,
|
meta.ReconcilingCondition,
|
||||||
},
|
},
|
||||||
NegativePolarity: []string{
|
NegativePolarity: []string{
|
||||||
sourcev1.BuildFailedCondition,
|
sourcev1.BuildFailedCondition,
|
||||||
sourcev1.FetchFailedCondition,
|
sourcev1.FetchFailedCondition,
|
||||||
sourcev1.ArtifactOutdatedCondition,
|
|
||||||
sourcev1.StorageOperationFailedCondition,
|
sourcev1.StorageOperationFailedCondition,
|
||||||
|
sourcev1.ArtifactOutdatedCondition,
|
||||||
meta.StalledCondition,
|
meta.StalledCondition,
|
||||||
meta.ReconcilingCondition,
|
meta.ReconcilingCondition,
|
||||||
},
|
},
|
||||||
|
|
|
@ -58,23 +58,23 @@ var helmRepositoryReadyCondition = summarize.Conditions{
|
||||||
Target: meta.ReadyCondition,
|
Target: meta.ReadyCondition,
|
||||||
Owned: []string{
|
Owned: []string{
|
||||||
sourcev1.FetchFailedCondition,
|
sourcev1.FetchFailedCondition,
|
||||||
sourcev1.ArtifactOutdatedCondition,
|
|
||||||
sourcev1.StorageOperationFailedCondition,
|
sourcev1.StorageOperationFailedCondition,
|
||||||
|
sourcev1.ArtifactOutdatedCondition,
|
||||||
meta.ReadyCondition,
|
meta.ReadyCondition,
|
||||||
meta.ReconcilingCondition,
|
meta.ReconcilingCondition,
|
||||||
meta.StalledCondition,
|
meta.StalledCondition,
|
||||||
},
|
},
|
||||||
Summarize: []string{
|
Summarize: []string{
|
||||||
sourcev1.FetchFailedCondition,
|
sourcev1.FetchFailedCondition,
|
||||||
sourcev1.ArtifactOutdatedCondition,
|
|
||||||
sourcev1.StorageOperationFailedCondition,
|
sourcev1.StorageOperationFailedCondition,
|
||||||
|
sourcev1.ArtifactOutdatedCondition,
|
||||||
meta.StalledCondition,
|
meta.StalledCondition,
|
||||||
meta.ReconcilingCondition,
|
meta.ReconcilingCondition,
|
||||||
},
|
},
|
||||||
NegativePolarity: []string{
|
NegativePolarity: []string{
|
||||||
sourcev1.FetchFailedCondition,
|
sourcev1.FetchFailedCondition,
|
||||||
sourcev1.ArtifactOutdatedCondition,
|
|
||||||
sourcev1.StorageOperationFailedCondition,
|
sourcev1.StorageOperationFailedCondition,
|
||||||
|
sourcev1.ArtifactOutdatedCondition,
|
||||||
meta.StalledCondition,
|
meta.StalledCondition,
|
||||||
meta.ReconcilingCondition,
|
meta.ReconcilingCondition,
|
||||||
},
|
},
|
||||||
|
|
Loading…
Reference in New Issue