Skip to content

[full ci] Revert govc changes and update event page value#6970

Closed
mdharamadas1 wants to merge 4 commits intovmware:masterfrom
mdharamadas1:revert-govc-eventpage-changes
Closed

[full ci] Revert govc changes and update event page value#6970
mdharamadas1 wants to merge 4 commits intovmware:masterfrom
mdharamadas1:revert-govc-eventpage-changes

Conversation

@mdharamadas1
Copy link
Contributor

@mdharamadas1 mdharamadas1 commented Dec 15, 2017

Revert "Increase event page size to 1000 (#6937)
Change event page size to 300
Revert "Integration image govc v0.16.0 (#6788)"
Add GitHub check to two failing vic machine tests

@mdharamadas1 mdharamadas1 changed the title Revert govc and event page changes [full ci] Revert govc and event page changes Dec 15, 2017
@mdharamadas1 mdharamadas1 force-pushed the revert-govc-eventpage-changes branch from 82b855a to 393540c Compare December 15, 2017 20:48
@cgtexmex
Copy link
Contributor

Why the revert?

@mdharamadas1
Copy link
Contributor Author

@cgtexmex Sorry, didn't mention earlier but this is just for testing purpose as requested by @mhagen-vmware

@cgtexmex
Copy link
Contributor

so why revert from master and pollute the git history? Can't we use a different branch? Or is this a CI / Testing limitation?

@mhagen-vmware
Copy link
Contributor

Not going to merge it, just testing

@andrewtchin
Copy link
Contributor

Merging this will break some of the tests in #6816 so those will need to be disabled or something

@mdharamadas1 mdharamadas1 force-pushed the revert-govc-eventpage-changes branch from 393540c to a17b64c Compare December 18, 2017 17:57
@mdharamadas1 mdharamadas1 changed the title [full ci] Revert govc and event page changes [full ci] Revert govc changes and update event page value Dec 18, 2017
@mdharamadas1
Copy link
Contributor Author

All, I have updated the description for this PR.

@mdharamadas1 mdharamadas1 force-pushed the revert-govc-eventpage-changes branch from a17b64c to b6ef0d6 Compare December 18, 2017 18:01
@hickeng
Copy link
Contributor

hickeng commented Dec 22, 2017

#6998 addressed this

@hickeng hickeng closed this Dec 22, 2017
@mdharamadas1 mdharamadas1 mentioned this pull request Jan 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants