Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sprint 16 Changelog updates #1142

Merged
merged 1 commit into from
Dec 4, 2014
Merged

Sprint 16 Changelog updates #1142

merged 1 commit into from
Dec 4, 2014

Conversation

mfeifer
Copy link
Contributor

@mfeifer mfeifer commented Dec 1, 2014

Updated links.
Added known bullet points.

[skip ci]

@mfeifer
Copy link
Contributor Author

mfeifer commented Dec 1, 2014

@Fryguy @jrafanie @blomquisg @dclarizio @roliveri @abellotti @kbrock . Please let me know what you need added/changed. 🌴

@tenderlove
Copy link
Contributor

Rails Fork removal:

  • Backport disable_ddl_transaction! from Rails master to our fork
  • Update the main app to use disable_ddl_transaction!
  • Add bigserial support for primary keys to Rails (including table creation and FK column creation)
  • Backport bigserial API to our fork
  • Update application to use new API

@abellotti
Copy link
Member

Rest API:

  • Added support for accounts sub-collection /api/vms/#/accounts
  • Added support for software sub-collection /api/vms/#/software

@Fryguy
Copy link
Member

Fryguy commented Dec 3, 2014

Looks good...there are some minor style things...

  • extra spaces after some :
  • extra unneeded blank lines
  • typo modelling should be modeling (yes, I know they're both acceptable but 'MURICAN ENGLISH)
  • vmware, rhevm, ovirt should be capitalized properly -> VmWare VMware, RHEVM, oVirt
  • Title case sentences should be made into sentence case, e.g. "Specify Zone for Web Service Automation Request" -> "Specify zone for web service automation request"
  • "Default Group for LDAP Authentication" stuff an Added or Changed? I thought it was added, but perhaps not? @gmcculloug ?
  • SmartProxy directory was host, not miqhost
  • Might want to state these notable fixes:
### [Fixes](...)

- 44 issues fixed. Notable fixes include
  - Fix issue where deleting a Cluster or Host tries to delete all policy_events, thus never completing when there are millions of events.
  - Fix inheriting tags from resource pool.
  - Fix openstack provisioning to deal with multiple security groups with the same name.
  - Fix seeding of VmdbDatabase timing out with millions of vmdb_metrics rows

@jerryk55
Copy link
Member

jerryk55 commented Dec 3, 2014

@Fryguy if you're gonna be pedantic (like I am) "VMware" is the proper capitalization.

@Fryguy
Copy link
Member

Fryguy commented Dec 4, 2014

Thanks @jerryk55 ! Yes, I meant VMware (I always mix up their capitalization)

- Fixed issue where deleting a cluster or host tries to delete all policy_events, thus never completing when there are millions of events.
- Fixed inheriting tags from resource pool.
- Fixed openstack provisioning to deal with multiple security groups with the same name.
- Fixed seeding of VmdbDatabase timing out with millions of vmdb_metrics rows
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry to be a pain, but there should be 2 spaces before these bullets

Updated links.
Added known bullet points.
Added bullets from sprint report.
Removed extra line and character spaces.
Made sentence case corrections.
Added notable fixes

[skip ci]
@miq-bot
Copy link
Member

miq-bot commented Dec 4, 2014

Checked commit mfeifer@d544c23 with rubocop 0.27.1
0 files checked, 0 offenses detected
Everything looks good. 🍪

Fryguy added a commit that referenced this pull request Dec 4, 2014
Sprint 16 Changelog updates
@Fryguy Fryguy merged commit e9e0194 into ManageIQ:master Dec 4, 2014
@Fryguy Fryguy added this to the Sprint 16 Ending Dec 2, 2014 milestone Dec 4, 2014
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.

7 participants